Will Berkeley has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8341
Change subject: KUDU-2170 Masters can start with duplicates specified ...................................................................... KUDU-2170 Masters can start with duplicates specified Users can specify duplicate masters in --master_addresses, where a duplicate includes duplicated names and the same server known by different names. This doesn't cause problems as far as I know, but it looks scary because, for example, the web ui shows extra masters listed in /masters. This is because /masters shows one master per result from ListMasters. This patch deduplicates the output of ListMasters by UUID. It also warns on start if there are exact string duplicates in --master_addresses, at the time the master compares on-disk and configured master addresses, but before the masters are ready to resolve UUIDs. Also, the error message when on-disk and configured masters don't match is confusing. This patch makes it easy to understand. Change-Id: I5047ccf92dde5377da18a1a84e105a951fa5b48c --- M src/kudu/master/master.cc M src/kudu/master/master.h M src/kudu/master/sys_catalog.cc M src/kudu/master/sys_catalog.h 4 files changed, 22 insertions(+), 5 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/41/8341/1 -- To view, visit http://gerrit.cloudera.org:8080/8341 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I5047ccf92dde5377da18a1a84e105a951fa5b48c Gerrit-Change-Number: 8341 Gerrit-PatchSet: 1 Gerrit-Owner: Will Berkeley <wdberke...@gmail.com>
