Abhishek Chennaka has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17141
Change subject: KUDU-3226 Validate List of Masters for kudu ksck ...................................................................... KUDU-3226 Validate List of Masters for kudu 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. Change-Id: I2f3b2b7dcf2ac78cb95cf43242651e3ce8fddf6f --- M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/tool_action_common.cc 2 files changed, 28 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/41/17141/1 -- To view, visit http://gerrit.cloudera.org:8080/17141 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I2f3b2b7dcf2ac78cb95cf43242651e3ce8fddf6f Gerrit-Change-Number: 17141 Gerrit-PatchSet: 1 Gerrit-Owner: Abhishek Chennaka <[email protected]>
