-
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
schedule: fix scatter range #1642
Conversation
Can we improve the metrics? now, it shares the same metrics with others. |
@nolouch The way I used is not elegant. I am not sure if there is a better to implement it. |
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: Ryan Leung <rleungx@gmail.com>
Signed-off-by: Ryan Leung <rleungx@gmail.com>
Signed-off-by: Ryan Leung <rleungx@gmail.com>
019bfbf
to
9ee518f
Compare
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
/run-integration-ddl-test |
1 similar comment
/run-integration-ddl-test |
* fix scatter range Signed-off-by: Ryan Leung <rleungx@gmail.com> * split scatter range metrics Signed-off-by: Ryan Leung <rleungx@gmail.com>
* fix scatter range Signed-off-by: Ryan Leung <rleungx@gmail.com> * split scatter range metrics Signed-off-by: Ryan Leung <rleungx@gmail.com>
What problem does this PR solve?
There are some mistakes created by the previous PR which causes the scatter range scheduler cannot work.
What is changed and how it works?
This PR fixes it to make it work like before.
Check List
Tests
Related changes