-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
HBASE-26117 RIT counts should be configured when cluster is balancing #3524
base: master
Are you sure you want to change the base?
Conversation
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
This is hard coded in the past? Oh no. And maybe it should be a percentage instead of a fixed value? Consider you have 10 regions, maybe 5 is too large but if you have 100k regions, 5 is too small? WDYT? |
@Apache9 |
Checking the code again, I think here, the hard coded 5 is just for printing logs, not for the actual balancing work. Could you please confirm this again?
The code is like this, we never use the toPrint list in other places. Thanks. |
No description provided.