Hello Jean-Daniel Cryans, Mike Percy, Adar Dembo,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/2795
to review the following change.
Change subject: KUDU-1410 (part 2). rpc: clean up generated RPC service code
......................................................................
KUDU-1410 (part 2). rpc: clean up generated RPC service code
This changes the generated RPC service code to extract the actual
handling logic into a non-generated base class. The generated
code now just generates a map of method name to MethodInfo, each
of which contains the requisite information to handle an RPC.
This results in a small reduction in lines of non-generated code,
and a larger reduction in lines of generated code.
Furthermore, this refactor makes it easier to make improvements and
further cleanups on the RPC server side, since most changes now
won't need to affect the code generator.
Change-Id: Id7b42a5ba1de59315660a79b125e70938c3a1b2e
---
M src/kudu/rpc/protoc-gen-krpc.cc
M src/kudu/rpc/service_if.cc
M src/kudu/rpc/service_if.h
3 files changed, 90 insertions(+), 90 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/95/2795/1
--
To view, visit http://gerrit.cloudera.org:8080/2795
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id7b42a5ba1de59315660a79b125e70938c3a1b2e
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Mike Percy <[email protected]>
