-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
*: introduce snapshot into analyze #24575
Conversation
23fdfec
to
544a0a8
Compare
/run-check_dev_2 |
1bed6d2
to
9b4a63d
Compare
/run-check_dev_2 |
[REVIEW NOTIFICATION] This pull request has been approved by:
To complete the pull request process, please ask the reviewers in the list to review by filling The full list of commands accepted by this bot can be found here. Reviewer can indicate their review by submitting an approval review. |
/merge |
This pull request has been accepted and is ready to merge. Commit hash: cad0bc6
|
What problem does this PR solve?
First PR to fix #23453
Problem Summary:
modify_count is not as expected.
What is changed and how it works?
Proposal: RFC for a More Accurate ModifyCount
What's Changed:
snapshot
intomysql.stats_meta
AnalyzeResults
to wrapAnalyzeResult
of a same tablemysql.stats_meta
once in the analyzeHow it Works:
Related changes
N/A
Check List
Tests
Side effects
N/A
Release note