Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17995 )
Change subject: [master] KUDU-3311 Allow to start with diff num of masters ...................................................................... [master] KUDU-3311 Allow to start with diff num of masters Modified SysCatalogTable:Load to allow cluster startup if +1 master address is present in the flags. The startup is halted, if the number of masters on-disk is higher than what was provided in the flags, discouraging decommissioning masters this way. Additional manual test was done with the following steps in order to ensure, that the documented way of removing a master works: - ./start_kudu.sh -m 4 -t 3 -b $KUDU_BUILD_DIR -c /tmp/data --rpc-master 8800 - looked up the "last" master which was listening on :8006 and removed it by running: ./kudu master remove 127.0.0.1:8800,127.0.0.1:8802,127.0.0.1:8804,127.0.0.1:8806 127.0.0.1:8806 - stopped the cluster by running ./stop_kudu.sh - restarted with 3 masters by running ./start_kudu.sh -m 3 -t 3 -b $KUDU_BUILD_DIR -c /tmp/data --rpc-master 8800 Change-Id: I39aeee2f52a55a8c29770f748895d38c9adff8a2 Reviewed-on: http://gerrit.cloudera.org:8080/17995 Reviewed-by: Andrew Wong <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/integration-tests/master_replication-itest.cc M src/kudu/master/dynamic_multi_master-test.cc M src/kudu/master/master_options-test.cc M src/kudu/master/sys_catalog.cc M src/kudu/mini-cluster/external_mini_cluster.h 5 files changed, 141 insertions(+), 15 deletions(-) Approvals: Andrew Wong: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/17995 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I39aeee2f52a55a8c29770f748895d38c9adff8a2 Gerrit-Change-Number: 17995 Gerrit-PatchSet: 16 Gerrit-Owner: Zoltan Chovan <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Zoltan Chovan <[email protected]>
