You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ga...@apache.org on 2011/11/01 18:47:41 UTC

svn commit: r1196163 - in /hbase/branches/0.92: CHANGES.txt src/main/java/org/apache/hadoop/hbase/ipc/HBaseRPC.java

Author: garyh
Date: Tue Nov  1 17:47:41 2011
New Revision: 1196163

URL: http://svn.apache.org/viewvc?rev=1196163&view=rev
Log:
HBASE-4710  UnknownProtocolException should abort client retries as a DoNotRetryIOException

Modified:
    hbase/branches/0.92/CHANGES.txt
    hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/ipc/HBaseRPC.java

Modified: hbase/branches/0.92/CHANGES.txt
URL: http://svn.apache.org/viewvc/hbase/branches/0.92/CHANGES.txt?rev=1196163&r1=1196162&r2=1196163&view=diff
==============================================================================
--- hbase/branches/0.92/CHANGES.txt (original)
+++ hbase/branches/0.92/CHANGES.txt Tue Nov  1 17:47:41 2011
@@ -405,6 +405,7 @@ Release 0.92.0 - Unreleased
    HBASE-4651  ConcurrentModificationException might be thrown in
                TestHCM.testConnectionUniqueness (Jinchao)
    HBASE-4690  Intermittent TestRegionServerCoprocessorExceptionWithAbort failure
+   HBASE-4710  UnknownProtocolException should abort client retries
 
   IMPROVEMENTS
    HBASE-3290  Max Compaction Size (Nicolas Spiegelberg via Stack)  

Modified: hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/ipc/HBaseRPC.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/ipc/HBaseRPC.java?rev=1196163&r1=1196162&r2=1196163&view=diff
==============================================================================
--- hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/ipc/HBaseRPC.java (original)
+++ hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/ipc/HBaseRPC.java Tue Nov  1 17:47:41 2011
@@ -185,6 +185,11 @@ public class HBaseRPC {
   public static class UnknownProtocolException extends DoNotRetryIOException {
     private Class<?> protocol;
 
+    public UnknownProtocolException(String mesg) {
+      // required for unwrapping from a RemoteException
+      super(mesg);
+    }
+
     public UnknownProtocolException(Class<?> protocol) {
       this(protocol, "Server is not handling protocol "+protocol.getName());
     }