-
Notifications
You must be signed in to change notification settings - Fork 721
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
Add leader safeguard filter for balance-leader-scheduler (#2617) #2636
Conversation
/merge |
Your auto merge job has been accepted, waiting for:
|
/run-all-tests |
Codecov Report
@@ Coverage Diff @@
## release-4.0 #2636 +/- ##
===============================================
+ Coverage 77.11% 77.13% +0.02%
===============================================
Files 205 205
Lines 22115 22151 +36
===============================================
+ Hits 17053 17087 +34
+ Misses 3761 3760 -1
- Partials 1301 1304 +3
Continue to review full report at Codecov.
|
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
/merge |
Your auto merge job has been accepted, waiting for:
|
/run-all-tests |
@Yisaer merge failed. |
/merge |
Your auto merge job has been accepted, waiting for:
|
/merge |
Your auto merge job has been accepted, waiting for:
|
/run-all-tests |
@Yisaer merge failed. |
/run-all-tests |
@Yisaer merge failed. |
/merge |
/run-all-tests |
What problem does this PR solve?
close #2607
What is changed and how it works?
Add filter to check whether the target store break the placement rules in balance-leader-schelder in transferLeaderOut case and transferLeaderIn case.
Check List
Tests
Release note