Todd Lipcon has submitted this change and it was merged. Change subject: java: use a netty frame decoder instead of replaying decoder ......................................................................
java: use a netty frame decoder instead of replaying decoder All of our inbound packets are length-prefixed. Netty provides a nice class for handing length-prefixed messages. This avoids a bunch of more custom logic we were doing for length checking, etc. Change-Id: I1ec1e6f9cbbdf694fe85b7668a3c349efc26c08f Reviewed-on: http://gerrit.cloudera.org:8080/5926 Reviewed-by: Dan Burkert <[email protected]> Reviewed-by: Jean-Daniel Cryans <[email protected]> Tested-by: Kudu Jenkins --- M java/kudu-client/src/main/java/org/apache/kudu/client/CallResponse.java M java/kudu-client/src/main/java/org/apache/kudu/client/ConnectionCache.java M java/kudu-client/src/main/java/org/apache/kudu/client/KuduRpc.java M java/kudu-client/src/main/java/org/apache/kudu/client/SecureRpcHelper.java M java/kudu-client/src/main/java/org/apache/kudu/client/TabletClient.java 5 files changed, 48 insertions(+), 122 deletions(-) Approvals: Dan Burkert: Looks good to me, approved Jean-Daniel Cryans: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/5926 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1ec1e6f9cbbdf694fe85b7668a3c349efc26c08f Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
