Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5946 to look at the new patch set (#2). Change subject: [rpc] Assume existence of TlsContext during negotiation ...................................................................... [rpc] Assume existence of TlsContext during negotiation This changes the client and server RPC negotiatiators to always assume the existence of a TlsContext for the messenger, and check whether there are valid certs in the context in order to decide whether to use TLS encryption. This will simplify authentication negotiation down the line. Change-Id: I806ca360bcd168b1ab075a4c7143436a7ccbba06 --- 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, 74 insertions(+), 66 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/46/5946/2 -- To view, visit http://gerrit.cloudera.org:8080/5946 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I806ca360bcd168b1ab075a4c7143436a7ccbba06 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org>