Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8393 )
Change subject: KUDU-2200: provide better diagnostics when connecting to a subset of masters ...................................................................... KUDU-2200: provide better diagnostics when connecting to a subset of masters This changes the ConnectToMaster RPC to send back the list of the master peers, and then changes the clients to use this information to provide better error messages in the case that the user has specified only a subset of the live masters. Note that this patch follows a "first do no harm" philosophy. In the case that the user "gets lucky" and only specifies one of their three masters, and that master happens to be the leader, we'll continue to treat that as "successful". Change-Id: I52f903e1aa5ae6948ca1ba6d4d856c3c9dc73d56 Reviewed-on: http://gerrit.cloudera.org:8080/8393 Reviewed-by: Alexey Serbin <aser...@cloudera.com> Tested-by: Kudu Jenkins --- M java/kudu-client/src/main/java/org/apache/kudu/client/ConnectToCluster.java M java/kudu-client/src/main/java/org/apache/kudu/client/ProtobufHelper.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestConnectToCluster.java M src/kudu/client/master_rpc.cc M src/kudu/integration-tests/master_replication-itest.cc M src/kudu/master/master-test.cc M src/kudu/master/master.cc M src/kudu/master/master.h M src/kudu/master/master.proto M src/kudu/master/master_service.cc 10 files changed, 256 insertions(+), 9 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8393 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I52f903e1aa5ae6948ca1ba6d4d856c3c9dc73d56 Gerrit-Change-Number: 8393 Gerrit-PatchSet: 4 Gerrit-Owner: Todd Lipcon <t...@apache.org> 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>