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/07/11 20:19:38 UTC

svn commit: r1502318 - /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java

Author: jxiang
Date: Thu Jul 11 18:19:37 2013
New Revision: 1502318

URL: http://svn.apache.org/r1502318
Log:
HBASE-8899 Could not open a region on a server where it is being closed - ADDENDUM

Modified:
    hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java

Modified: hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java?rev=1502318&r1=1502317&r2=1502318&view=diff
==============================================================================
--- hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java (original)
+++ hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java Thu Jul 11 18:19:37 2013
@@ -3419,7 +3419,8 @@ public class HRegionServer implements Cl
             // be true or false. True means the region is opening on this RS; while false
             // means the region is closing. Only return ALREADY_OPENED if not closing (i.e.
             // not in transition any more, or still transition to open.
-            if (!Boolean.FALSE.equals(closing)) {
+            if (!Boolean.FALSE.equals(closing)
+                && getFromOnlineRegions(region.getEncodedName()) != null) {
               LOG.warn("Attempted open of " + region.getEncodedName()
                 + " but already online on this server");
               builder.addOpeningState(RegionOpeningState.ALREADY_OPENED);