You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by sy...@apache.org on 2016/02/09 00:13:48 UTC

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

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

This reverts commit c177cfed416e0cf72ae6131c5e98c99672bef3cc.


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

Branch: refs/heads/hbase-12439
Commit: 337f4830770d87e2fe629d742a3dcc64dba0c2cb
Parents: c177cfe
Author: stack <st...@apache.org>
Authored: Thu Feb 4 16:08:41 2016 -0800
Committer: stack <st...@apache.org>
Committed: Thu Feb 4 16:08:41 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/337f4830/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 8680c89..20f4169 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
@@ -816,11 +816,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;
       }
     }
@@ -843,11 +841,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;