Hello Tidy Bot, Alexey Serbin, Attila Bukor, Kudu Jenkins, Andrew Wong, Bankim
Bhavsar,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/17995
to look at the new patch set (#10).
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
---
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, 150 insertions(+), 15 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/95/17995/10
--
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: newpatchset
Gerrit-Change-Id: I39aeee2f52a55a8c29770f748895d38c9adff8a2
Gerrit-Change-Number: 17995
Gerrit-PatchSet: 10
Gerrit-Owner: Anonymous Coward <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Anonymous Coward <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Bankim Bhavsar <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
