You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2016/02/05 01:10:02 UTC

hbase git commit: Revert "HBASE-15120 Undo aggressive load balancer logging at tens of lines per millisecond"

Repository: hbase
Updated Branches:
  refs/heads/branch-1 66da601cd -> 8061fa4af


Revert "HBASE-15120 Undo aggressive load balancer logging at tens of lines per millisecond"

This reverts commit 66da601cdcc915c3ccebd994105c8f7fbebf556d.


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/8061fa4a
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/8061fa4a
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/8061fa4a

Branch: refs/heads/branch-1
Commit: 8061fa4af8adb512e5c61374fdef8870e94d281a
Parents: 66da601
Author: stack <st...@apache.org>
Authored: Thu Feb 4 16:09:52 2016 -0800
Committer: stack <st...@apache.org>
Committed: Thu Feb 4 16:09:52 2016 -0800

----------------------------------------------------------------------
 .../hadoop/hbase/master/balancer/BaseLoadBalancer.java       | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/8061fa4a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java
index 97494f0..3728c9a 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java
@@ -814,11 +814,9 @@ public abstract class BaseLoadBalancer implements LoadBalancer {
           i++;
           lowestLocalityServerIndex = serverIndicesSortedByLocality[i];
         }
-        if (LOG.isTraceEnabled()) {
-          LOG.trace("Lowest locality region server with non zero regions is "
+        LOG.debug("Lowest locality region server with non zero regions is "
             + servers[lowestLocalityServerIndex].getHostname() + " with locality "
             + localityPerServer[lowestLocalityServerIndex]);
-        }
         return lowestLocalityServerIndex;
       }
     }
@@ -841,11 +839,9 @@ public abstract class BaseLoadBalancer implements LoadBalancer {
             lowestLocalityRegionIndex = j;
           }
         }
-        if (LOG.isTraceEnabled()) {
-          LOG.debug(" Lowest locality region index is " + lowestLocalityRegionIndex
+        LOG.debug(" Lowest locality region index is " + lowestLocalityRegionIndex
             + " and its region server contains " + regionsPerServer[serverIndex].length
             + " regions");
-        }
         return regionsPerServer[serverIndex][lowestLocalityRegionIndex];
       } else {
         return -1;