You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by jx...@apache.org on 2014/02/03 21:37:14 UTC
svn commit: r1564048 -
/hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java
Author: jxiang
Date: Mon Feb 3 20:37:14 2014
New Revision: 1564048
URL: http://svn.apache.org/r1564048
Log:
HBASE-10456 Balancer should not run if it's just turned off
Modified:
hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java
Modified: hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java?rev=1564048&r1=1564047&r2=1564048&view=diff
==============================================================================
--- hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java (original)
+++ hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java Mon Feb 3 20:37:14 2014
@@ -1419,12 +1419,12 @@ MasterServices, Server {
LOG.debug("Master has not been initialized, don't run balancer.");
return false;
}
- // If balance not true, don't run balancer.
- if (!this.loadBalancerTracker.isBalancerOn()) return false;
// Do this call outside of synchronized block.
int maximumBalanceTime = getBalancerCutoffTime();
boolean balancerRan;
synchronized (this.balancer) {
+ // If balance not true, don't run balancer.
+ if (!this.loadBalancerTracker.isBalancerOn()) return false;
// Only allow one balance run at at time.
if (this.assignmentManager.getRegionStates().isRegionsInTransition()) {
Map<String, RegionState> regionsInTransition =