Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SERVER-78113: not cache meaningless plan, it will increased memory overhead and increase computing overhead. #1555

Open
wants to merge 1 commit into
base: v5.0
Choose a base branch
from

Conversation

y123456yz
Copy link
Contributor

// when the mode is PlanCachingMode::AlwaysCache or PlanCachingMode::SometimesCache, there is a special case.
// take Classic plan cache for example:
//   If the winning index's advanced = 0 && works == 1 && isEOF == true, which means we didn't get any data to rank score, all candidates 
//   score is same. the score of the calculation is meaningless.
//
//In this case, if we cache the plan, it will increased memory overhead. when the cached plan stage pick best plan, 
//it will trigger replan. this will increase computing overhead.

@y123456yz
Copy link
Contributor Author

@yuanfang0818 yuanfang0818 changed the title not cache meaningless plan, it will increased memory overhead and increase computing overhead. SERVER-78113: not cache meaningless plan, it will increased memory overhead and increase computing overhead. Jun 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant