HBASE-13366 Throw DoNotRetryIOException instead of read only IOException 
(Shaohui Liu)


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/8d3e933d
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/8d3e933d
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/8d3e933d

Branch: refs/heads/branch-1.1
Commit: 8d3e933de8288704f5be9f5cc3234a72e7083a0e
Parents: 3eb5300
Author: Liu Shaohui <[email protected]>
Authored: Thu Apr 2 11:52:54 2015 +0800
Committer: Andrew Purtell <[email protected]>
Committed: Tue May 5 18:12:43 2015 -0700

----------------------------------------------------------------------
 .../main/java/org/apache/hadoop/hbase/regionserver/HRegion.java   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/8d3e933d/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
----------------------------------------------------------------------
diff --git 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
index d7626c3..0d5306e 100644
--- 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
+++ 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
@@ -74,6 +74,7 @@ import org.apache.hadoop.hbase.Cell;
 import org.apache.hadoop.hbase.CellScanner;
 import org.apache.hadoop.hbase.CellUtil;
 import org.apache.hadoop.hbase.CompoundConfiguration;
+import org.apache.hadoop.hbase.DoNotRetryIOException;
 import org.apache.hadoop.hbase.DroppedSnapshotException;
 import org.apache.hadoop.hbase.HBaseConfiguration;
 import org.apache.hadoop.hbase.HColumnDescriptor;
@@ -3523,7 +3524,7 @@ public class HRegion implements HeapSize, 
PropagatingConfigurationObserver, Regi
    */
   protected void checkReadOnly() throws IOException {
     if (isReadOnly()) {
-      throw new IOException("region is read only");
+      throw new DoNotRetryIOException("region is read only");
     }
   }
 

Reply via email to