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 2013/02/15 22:37:51 UTC

svn commit: r1446784 - /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java

Author: jxiang
Date: Fri Feb 15 21:37:51 2013
New Revision: 1446784

URL: http://svn.apache.org/r1446784
Log:
HBASE-7799 reassigning region stuck in open still may not work correctly due to leftover ZK node

Modified:
    hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java

Modified: hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
URL: http://svn.apache.org/viewvc/hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java?rev=1446784&r1=1446783&r2=1446784&view=diff
==============================================================================
--- hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java (original)
+++ hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java Fri Feb 15 21:37:51 2013
@@ -2806,6 +2806,12 @@ public class AssignmentManager extends Z
             + " since it is not opening on the dead server any more: " + sn);
           it.remove();
         } else {
+          try{
+            // Delete the ZNode if exists
+            ZKAssign.deleteNodeFailSilent(watcher, hri);
+          } catch (KeeperException ke) {
+            server.abort("Unexpected ZK exception deleting node " + hri, ke);
+          }
           // Mark the region closed and assign it again by SSH
           regionStates.updateRegionState(hri, RegionState.State.CLOSED);
         }