Index: hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java =================================================================== --- hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java (revision 124855) +++ hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java (working copy) @@ -390,7 +390,7 @@ float average = cs.getLoadAverage(); // for logging int floor = (int) Math.floor(average * (1 - slop)); int ceiling = (int) Math.ceil(average * (1 + slop)); - if (!(cs.getMinLoad() > ceiling || cs.getMaxLoad() < floor)) { + if (!(cs.getMaxLoad() > ceiling || cs.getMinLoad() < floor)) { NavigableMap> serversByLoad = cs.getServersByLoad(); if (LOG.isTraceEnabled()) { // If nothing to balance, then don't say anything unless trace-level logging. Index: hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/ClusterLoadState.java =================================================================== --- hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/ClusterLoadState.java (revision 124855) +++ hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/ClusterLoadState.java (working copy) @@ -75,11 +75,11 @@ return (float) numRegions / numServers; } - int getMinLoad() { + int getMaxLoad() { return getServersByLoad().lastKey().getLoad(); } - int getMaxLoad() { + int getMinLoad() { return getServersByLoad().firstKey().getLoad(); }