Alexey Serbin has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/17216 )
Change subject: [rpc] minor cleanup on messenger.{h,cc}
......................................................................
[rpc] minor cleanup on messenger.{h,cc}
Minor code style-related clean-up on messenger.h and messenger.cc files:
* removed 'using ...' from messenger.h
* moved ampersand and asterisk to the type
* other nits
Change-Id: Id623a7d69c2d702cec3233d9282930e9bbd865ee
Reviewed-on: http://gerrit.cloudera.org:8080/17216
Tested-by: Alexey Serbin <[email protected]>
Reviewed-by: Andrew Wong <[email protected]>
---
M src/kudu/rpc/client_negotiation.cc
M src/kudu/rpc/client_negotiation.h
M src/kudu/rpc/messenger.cc
M src/kudu/rpc/messenger.h
M src/kudu/rpc/negotiation-test.cc
M src/kudu/rpc/negotiation.cc
M src/kudu/rpc/server_negotiation.cc
M src/kudu/rpc/server_negotiation.h
8 files changed, 117 insertions(+), 169 deletions(-)
Approvals:
Alexey Serbin: Verified
Andrew Wong: Looks good to me, approved
--
To view, visit http://gerrit.cloudera.org:8080/17216
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Id623a7d69c2d702cec3233d9282930e9bbd865ee
Gerrit-Change-Number: 17216
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
