Todd Lipcon has posted comments on this change. Change subject: TLS-negotiation [3/n]: rename negotiation protobuf messages ......................................................................
Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/5757/2/src/kudu/rpc/sasl_server.cc File src/kudu/rpc/sasl_server.cc: Line 176: TRACE("Waiting for next SASL message..."); next negotiation message? PS2, Line 200: SASL request. same PS2, Line 253: SaslMessage maybe this should be renamed? http://gerrit.cloudera.org:8080/#/c/5757/2/src/kudu/rpc/sasl_server.h File src/kudu/rpc/sasl_server.h: Line 122: // Encode and send the specified SASL message to the client. nit: this should say 'negotiation message' or something -- To view, visit http://gerrit.cloudera.org:8080/5757 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie24eec6e632c6e5064afa631eccb3877425fe354 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dan Burkert <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
