From 180c2d2391c132ad60356ed6ea81de492864c984 Mon Sep 17 00:00:00 2001 From: Benoit Sigoure Date: Sat, 20 Jul 2013 00:16:36 -0700 Subject: [PATCH] Don't require a cellBlockCodecClass in the RPC header. The protobuf definition defines one by default, so requiring one to be explicitly set is not necessary and violates the definition of the protobuf that claims it's got a default value. --- hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java | 1 - 1 file changed, 1 deletion(-) diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java index 25ca73c..c404d8d 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java @@ -1556,7 +1556,6 @@ public class RpcServer implements RpcServerInterface { private void setupCellBlockCodecs(final ConnectionHeader header) throws FatalConnectionException { // TODO: Plug in other supported decoders. - if (!header.hasCellBlockCodecClass()) throw new FatalConnectionException("No codec"); String className = header.getCellBlockCodecClass(); try { this.codec = (Codec)Class.forName(className).newInstance(); -- 1.8.2.1.539.g4196a96