-
Notifications
You must be signed in to change notification settings - Fork 722
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
*: make hot region scheduler configurable #1412
Conversation
Signed-off-by: nolouch <nolouch@gmail.com>
Codecov Report
@@ Coverage Diff @@
## master #1412 +/- ##
=========================================
+ Coverage 67.52% 67.62% +0.1%
=========================================
Files 157 157
Lines 14774 14800 +26
=========================================
+ Hits 9976 10009 +33
+ Misses 3874 3872 -2
+ Partials 924 919 -5
Continue to review full report at Codecov.
|
Signed-off-by: nolouch <nolouch@gmail.com>
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.
The rest LGTM.
Seems this is a good case for a Schrodinger test :) |
Signed-off-by: nolouch <nolouch@gmail.com>
/run-all-tests |
* *: make hot region scheduler configurable Signed-off-by: nolouch <nolouch@gmail.com>
*: make hot region scheduler configurable (#1412) Signed-off-by: nolouch <nolouch@gmail.com> * use high priority for the hot region related operators (#1492) Signed-off-by: Ryan Leung <rleungx@gmail.com> * schedulers: let hot region balance not affect by balance-region-scheduler-limit (#1522) Signed-off-by: nolouch <nolouch@gmail.com>
Signed-off-by: nolouch nolouch@gmail.com
What problem does this PR solve?
In some situations, the hot region scheduling is too intense and affects the latency jitter.
What is changed and how it works?
let something in the hot region schedule be configurable.
Check List
Tests