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/03/20 02:52:54 UTC

svn commit: r1083358 - /hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/handler/OpenedRegionHandler.java

Author: stack
Date: Sun Mar 20 01:52:54 2011
New Revision: 1083358

URL: http://svn.apache.org/viewvc?rev=1083358&view=rev
Log:
HBASE-3671) Split report before we finish parent region open; workaround till 0.92; Race between split and OPENED processing -- fix suggested by Ted Yu

Modified:
    hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/handler/OpenedRegionHandler.java

Modified: hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/handler/OpenedRegionHandler.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/handler/OpenedRegionHandler.java?rev=1083358&r1=1083357&r2=1083358&view=diff
==============================================================================
--- hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/handler/OpenedRegionHandler.java (original)
+++ hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/master/handler/OpenedRegionHandler.java Sun Mar 20 01:52:54 2011
@@ -93,7 +93,7 @@ public class OpenedRegionHandler extends
     }
     // Code to defend against case where we get SPLIT before region open
     // processing completes; temporary till we make SPLITs go via zk -- 0.92.
-    if (this.assignmentManager.isRegionInTransition(regionInfo) == null) {
+    if (this.assignmentManager.isRegionInTransition(regionInfo) != null) {
       this.assignmentManager.regionOnline(regionInfo, serverInfo);
     } else {
       LOG.warn("Skipping the onlining of " + regionInfo.getRegionNameAsString() +