Todd Lipcon has submitted this change and it was merged. Change subject: master_rpc: pass back more details from ConnectToCluster ......................................................................
master_rpc: pass back more details from ConnectToCluster This plumbs back the ConnectToMasterResponsePB from the ConnectToCluster RPC. This brings us one step closer to inserting the CA cert into the TlsContext and passing the token somewhere useful. Along the way, I switched to std::function instead of gutil Callback since it allows the slightly easier lambda syntax. No new tests here since all the existing client tests cover this code path. Change-Id: I3b5056c9f31237249516a2e7ae68d641c9f6bd02 Reviewed-on: http://gerrit.cloudera.org:8080/5892 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <aser...@cloudera.com> --- M src/kudu/client/client-internal.cc M src/kudu/client/client-internal.h M src/kudu/client/master_rpc.cc M src/kudu/client/master_rpc.h M src/kudu/integration-tests/external_mini_cluster.cc 5 files changed, 63 insertions(+), 56 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/5892 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3b5056c9f31237249516a2e7ae68d641c9f6bd02 Gerrit-PatchSet: 7 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org>