siddhantsangwan commented on pull request #2808: URL: https://github.com/apache/ozone/pull/2808#issuecomment-966897707
Also, please note the following concern that was raised [here](https://github.com/apache/ozone/pull/2230#discussion_r630705685) for an earlier version of balancer. > Question: > > say we have a 10 DN cluster, the usage of all of them is 95%, then one empty DN is added to rebalance the cluster. Given the threshold is 10%, it seems the balancer will not work in this case, since that 10 DN will not achieve upperLimit. > > Have we consider corner case like this ? By removing `withinThresholdUtilizedNodes`, we rely on the user setting a suitable threshold to make balancing work in such cases. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
