svn commit: r1138219 - in /hbase/trunk: CHANGES.txt src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java

[jdcryans]

Author: jdcryans
Date: Tue Jun 21 22:22:33 2011
New Revision: 1138219

URL: http://svn.apache.org/viewvc?rev=1138219&view=rev
Log:
   HBASE-3994  SplitTransaction has a window where clients can
               get RegionOfflineException

Modified:
    hbase/trunk/CHANGES.txt
    
hbase/trunk/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java

Modified: hbase/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hbase/trunk/CHANGES.txt?rev=1138219&r1=1138218&r2=1138219&view=diff
==============================================================================
--- hbase/trunk/CHANGES.txt (original)
+++ hbase/trunk/CHANGES.txt Tue Jun 21 22:22:33 2011
@@ -269,6 +269,8 @@ Release 0.91.0 - Unreleased
    HBASE-3789  Cleanup the locking contention in the master
    HBASE-3927  Display total uncompressed byte size of a region in web UI
    HBASE-4011  New MasterObserver hook: post startup of active master
+   HBASE-3994  SplitTransaction has a window where clients can
+               get RegionOfflineException
 
   TASKS
    HBASE-3559  Move report of split to master OFF the heartbeat channel

Modified: 
hbase/trunk/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java
URL: 
http://svn.apache.org/viewvc/hbase/trunk/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java?rev=1138219&r1=1138218&r2=1138219&view=diff
==============================================================================
--- 
hbase/trunk/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java
 (original)
+++ 
hbase/trunk/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java
 Tue Jun 21 22:22:33 2011
@@ -892,8 +892,15 @@ public class HConnectionManager {
             throw new TableNotFoundException(
               "Table '" + Bytes.toString(tableName) + "' was not found.");
           }
+          if (regionInfo.isSplit()) {
+            throw new RegionOfflineException("the only available region for" +
+              " the required row is a split parent," +
+              " the daughters should be online soon: " +
+              regionInfo.getRegionNameAsString());
+          }
           if (regionInfo.isOffline()) {
-            throw new RegionOfflineException("region offline: " +
+            throw new RegionOfflineException("the region is offline, could" +
+              " be caused by a disable table call: " +
               regionInfo.getRegionNameAsString());
           }