svn commit: r1041204 - in /hbase/trunk: CHANGES.txt src/main/java/org/apache/hadoop/hbase/regionserver/handler/OpenRegionHandler.java

[st...@apache.org]

Author: stack
Date: Wed Dec  1 22:56:27 2010
New Revision: 1041204

URL: http://svn.apache.org/viewvc?rev=1041204&view=rev
Log:
HBASE-3299  If failed open, we don't output the IOE

Modified:
    hbase/trunk/CHANGES.txt
    hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/handler/OpenRegionHandler.java

Modified: hbase/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hbase/trunk/CHANGES.txt?rev=1041204&r1=1041203&r2=1041204&view=diff
==============================================================================
--- hbase/trunk/CHANGES.txt (original)
+++ hbase/trunk/CHANGES.txt Wed Dec  1 22:56:27 2010
@@ -731,6 +731,7 @@ Release 0.90.0 - Unreleased
                of .META.
    HBASE-3294  WARN org.apache.hadoop.hbase.regionserver.Store: Not in set
                (double-remove?) org.apache.hadoop.hbase.regionserver.StoreScanner@76607d3d
+   HBASE-3299   If failed open, we don't output the IOE
 
 
   IMPROVEMENTS

Modified: hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/handler/OpenRegionHandler.java
URL: http://svn.apache.org/viewvc/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/handler/OpenRegionHandler.java?rev=1041204&r1=1041203&r2=1041204&view=diff
==============================================================================
--- hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/handler/OpenRegionHandler.java (original)
+++ hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/handler/OpenRegionHandler.java Wed Dec  1 22:56:27 2010
@@ -111,8 +111,8 @@ public class OpenRegionHandler extends E
           }
         });
     } catch (IOException e) {
-      LOG.error("IOException instantiating region for " + regionInfo +
-        "; resetting state of transition node from OPENING to OFFLINE");
+      LOG.error("Failed open of " + regionInfo +
+        "; resetting state of transition node from OPENING to OFFLINE", e);
       try {
         // TODO: We should rely on the master timing out OPENING instead of this
         // TODO: What if this was a split open?  The RS made the OFFLINE
@@ -216,4 +216,4 @@ public class OpenRegionHandler extends E
     }
     return openingVersion;
   }
-}
\ No newline at end of file
+}