Abhishek Chennaka has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17149
Change subject: KUDU-3226 Validate List of Masters for the tool kudu cluster ksck ...................................................................... KUDU-3226 Validate List of Masters for the tool kudu cluster ksck This patch checks for duplicate kudu masters reported in kudu cluster ksck/rebalance commands. It only does a string comparision check to detect and report all the duplicate master(/s). If a user provides the IP address and hostname of the same master, it won't be able to catch the duplicates. That seems more of a deliberate attempt rather than a common mistake a user could make and hence not considering that case. Also wrote a simple test case to test this functionality. Decided not to spin up a test cluster for this as this is more of client side check and a master is not actually contacted. This change got conflicted with another test i.e. AdminCliTest.TestAuthzResetCacheIncorrectMasterAddressList. Updated this test as well to by creating a new external mini cluster with three masters and running the command in the test with just one master Change-Id: Iaeaa3fc455b3484ff1a5f40e94c0071cd638b47c --- M src/kudu/tools/kudu-admin-test.cc M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/tool_action_common.cc 3 files changed, 24 insertions(+), 20 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/49/17149/1 -- To view, visit http://gerrit.cloudera.org:8080/17149 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Iaeaa3fc455b3484ff1a5f40e94c0071cd638b47c Gerrit-Change-Number: 17149 Gerrit-PatchSet: 1 Gerrit-Owner: Abhishek Chennaka <achenn...@cloudera.com>