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 2011/01/05 22:08:13 UTC
svn commit: r1055640 - in /hbase/branches/0.90: CHANGES.txt
src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
Author: stack
Date: Wed Jan 5 21:08:13 2011
New Revision: 1055640
URL: http://svn.apache.org/viewvc?rev=1055640&view=rev
Log:
HBASE-3420 Handling a big rebalance, we can queue multiple instances of a Close event; messes up state
Modified:
hbase/branches/0.90/CHANGES.txt
hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
Modified: hbase/branches/0.90/CHANGES.txt
URL: http://svn.apache.org/viewvc/hbase/branches/0.90/CHANGES.txt?rev=1055640&r1=1055639&r2=1055640&view=diff
==============================================================================
--- hbase/branches/0.90/CHANGES.txt (original)
+++ hbase/branches/0.90/CHANGES.txt Wed Jan 5 21:08:13 2011
@@ -778,6 +778,8 @@ Release 0.90.0 - Unreleased
HBASE-3409 Failed server shutdown processing when retrying hlog split
HBASE-3410 Unable to set/modify TTL on a column family using the shell
HBASE-3412 HLogSplitter should handle missing HLogs
+ HBASE-3420 Handling a big rebalance, we can queue multiple instances
+ of a Close event; messes up state
IMPROVEMENTS
Modified: hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java?rev=1055640&r1=1055639&r2=1055640&view=diff
==============================================================================
--- hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java (original)
+++ hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java Wed Jan 5 21:08:13 2011
@@ -1567,11 +1567,13 @@ public class AssignmentManager extends Z
// Expired! Do a retry.
switch (regionState.getState()) {
case CLOSED:
- LOG.info("Region has been CLOSED for too long, " +
- "retriggering ClosedRegionHandler");
- AssignmentManager.this.executorService.submit(
- new ClosedRegionHandler(master, AssignmentManager.this,
- regionState.getRegion()));
+ LOG.info("Region " + regionInfo.getEncodedName() +
+ " has been CLOSED for too long, waiting on queued " +
+ "ClosedRegionHandler to run or server shutdown");
+ // Update our timestamp.
+ synchronized(regionState) {
+ regionState.update(regionState.getState());
+ }
break;
case OFFLINE:
LOG.info("Region has been OFFLINE for too long, " +