You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hbase.apache.org by mb...@apache.org on 2013/12/02 12:09:29 UTC

svn commit: r1546961 - /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java

Author: mbertozzi
Date: Mon Dec  2 11:09:29 2013
New Revision: 1546961

URL: http://svn.apache.org/r1546961
Log:
HBASE-9978 The client retries even if the method is not present on the server

Modified:
    hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java

Modified: hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java
URL: http://svn.apache.org/viewvc/hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java?rev=1546961&r1=1546960&r2=1546961&view=diff
==============================================================================
--- hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java (original)
+++ hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java Mon Dec  2 11:09:29 2013
@@ -1713,6 +1713,12 @@ public class RpcServer implements RpcSer
       } catch (Throwable t) {
         String msg = "Unable to read call parameter from client " + getHostAddress();
         LOG.warn(msg, t);
+
+        // If the method is not present on the server, do not retry.
+        if (t instanceof UnsupportedOperationException) {
+          t = new DoNotRetryIOException(t);
+        }
+
         final Call readParamsFailedCall =
           new Call(id, this.service, null, null, null, null, this,
             responder, totalRequestSize, null);