-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
tikv: make region_request_test stable #11541
Conversation
/run-all-tests |
Codecov Report
@@ Coverage Diff @@
## master #11541 +/- ##
===========================================
Coverage 81.2937% 81.2937%
===========================================
Files 425 425
Lines 91643 91643
===========================================
Hits 74500 74500
Misses 11788 11788
Partials 5355 5355 |
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
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
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
cherry pick to release-3.0 failed |
What problem does this PR solve?
old TestOnSendFailedWithCloseKnownStoreThenUseNewOne test is occasionally failed
the old test has a problem new store never be a leader and accept the request.
and old assert will be failed to because of async scheduleReload + race/leak slow running.
What is changed and how it works?
elect a new leader to another store when one store fails
Check List
Tests
Code changes
Side effects
Related changes
This change is