-
Notifications
You must be signed in to change notification settings - Fork 729
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 parameters more reasonable #3749
Labels
type/enhancement
The issue or PR belongs to an enhancement.
Comments
hey @nolouch, may I pick up this issue? |
And I think the limit here means the parameters for scheduler, is my understanding correct? |
@xuhui-lu Yes, you are right. such as |
I think you should decouple the merge-limit, region-limit, leader-limit first. |
There are some job to solve this problem:
|
2 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Feature Request
Describe your feature request related problem
We have encountered many problems caused by improper parameter adjustment. We must limit the adjustment range of the parameters and give a best practice, remove some useless and do some check.
Such as:
merge-limit=128, region-limit=64, leader-limit=8
the merge limit may influence the leader limit.Describe alternatives you've considered
store limit
should have best practiceTeachability, Documentation, Adoption, Migration Strategy
The text was updated successfully, but these errors were encountered: