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
raftstore: use new approximate keys function when result is 0 (#5403) #5414
Conversation
Signed-off-by: root <you1474600@gmail.com>
Signed-off-by: root <you1474600@gmail.com>
Signed-off-by: you06 <you1474600@gmail.com>
Signed-off-by: you06 <you1474600@gmail.com>
/run-all-tests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: root <you1474600@gmail.com>
/run-integration-cop-push-down-test |
/run-integration-cop-push-down-test tidb=release-3.0 pd=release-3.0 tidb-test=release-3.0 |
/run-integration-cop-push-down-test tidb=release-3.0 pd=release-3.0 tidb-test=master |
/integration-compatibility-test /integration-ddl-test |
/run-all-tests |
/run-integration-cop-push-down-test tidb=release-3.0 pd=release-3.0 tidb-test=master |
/run-integration-cop-push-down-test |
/run-integration-cop-push-down-test tidb-test=pr/883 |
2 similar comments
/run-integration-cop-push-down-test tidb-test=pr/883 |
/run-integration-cop-push-down-test tidb-test=pr/883 |
/run-integration-cop-push-down-test tidb=release-3.0 pd=release-3.0 tidb-test=pr/883 |
/run-integration-cop-push-down-test tidb-test=pr/883 |
1 similar comment
/run-integration-cop-push-down-test tidb-test=pr/883 |
cherry-pick #5403 to release-3.0
Signed-off-by: root you1474600@gmail.com
What have you changed?
Please explain in detail what the changes are in this PR and why they are needed:
Summarize your change
When the new
get_region_approximate_keys_cf
function return 0, theget_region_approximate_keys
function will downgrade to use the oldget_range_entries_and_versions
function. For this situation, 0 should be returned.Describe any limitations of the current code (optional)
Add a test but the the test also passed using old
get_range_entries_and_versions
function.What is the type of the changes?
Pick one of the following and delete the others:
How is the PR tested?
Add a test function
raftstore::coprocessor::split_check::keys::tests::test_region_approximate_keys_sub_region
.Does this PR affect documentation (docs) or should it be mentioned in the release notes?
No
Does this PR affect
tidb-ansible
?No
Refer to a related PR or issue link (optional)
#5306