hbase-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From la...@apache.org
Subject git commit: HBASE-12334 Handling of DeserializationException causes needless retry on failure.
Date Fri, 24 Oct 2014 23:23:37 GMT
Repository: hbase
Updated Branches:
  refs/heads/master 97acb9ef2 -> 6c7543c9c


HBASE-12334 Handling of DeserializationException causes needless retry on failure.


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

Branch: refs/heads/master
Commit: 6c7543c9c7d88b6d77e8d0d52ddad260fb487ae4
Parents: 97acb9e
Author: Lars Hofhansl <larsh@apache.org>
Authored: Fri Oct 24 16:21:33 2014 -0700
Committer: Lars Hofhansl <larsh@apache.org>
Committed: Fri Oct 24 16:21:33 2014 -0700

----------------------------------------------------------------------
 .../main/java/org/apache/hadoop/hbase/protobuf/ProtobufUtil.java | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/6c7543c9/hbase-client/src/main/java/org/apache/hadoop/hbase/protobuf/ProtobufUtil.java
----------------------------------------------------------------------
diff --git a/hbase-client/src/main/java/org/apache/hadoop/hbase/protobuf/ProtobufUtil.java
b/hbase-client/src/main/java/org/apache/hadoop/hbase/protobuf/ProtobufUtil.java
index c7d2b24..768da76 100644
--- a/hbase-client/src/main/java/org/apache/hadoop/hbase/protobuf/ProtobufUtil.java
+++ b/hbase-client/src/main/java/org/apache/hadoop/hbase/protobuf/ProtobufUtil.java
@@ -1447,7 +1447,9 @@ public final class ProtobufUtil {
       }
       return (Filter)parseFrom.invoke(c, value);
     } catch (Exception e) {
-      throw new IOException(e);
+      // Either we couldn't instantiate the method object, or "parseFrom" failed.
+      // In either case, let's not retry.
+      throw new DoNotRetryIOException(e);
     }
   }
 


Mime
View raw message