wchevreuil commented on code in PR #8218:
URL: https://github.com/apache/hbase/pull/8218#discussion_r3225040631


##########
hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/CacheAwareLoadBalancer.java:
##########


Review Comment:
   The CacheAwareLoadBalancer was originally thought for use cases where 
optimal cache allocation is pivotal for overall performance, so it only combine 
skewness and cache allocation costs when weighting in an imbalance. For use 
cases where locality/rack/load are more critical, the StochasticLoadBalancer 
should be used.



-- 
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]

Reply via email to