Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17010 )
Change subject: [tool] KUDU-2181 CLI to remove master ...................................................................... Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/17010/7/src/kudu/master/dynamic_multi_master-test.cc File src/kudu/master/dynamic_multi_master-test.cc: http://gerrit.cloudera.org:8080/#/c/17010/7/src/kudu/master/dynamic_multi_master-test.cc@874 PS7, Line 874: pda > It's not required because there is no ambiguity with other master. I'll add > it back for a positive case where uuid isn't strictly necessary. Done this part. > One way that comes to mind is to shutdown the master, delete the contents of > the master's directories, and restarting the same master. IIRC, that way the > master keeps its hostport, but assigns itself a new UUID. This is an interesting idea, so the deleted master still remains part of the Raft config and would be declared unavailable/unhealthy and a new master with same HostPort and different uuid shows up. I haven't tried it but I have some doubts, if the directory is deleted then how will the restarted master become part of Raft config. I'm going to table this particular test case of multiple masters with same HostPort for now. Filed KUDU-3246. -- To view, visit http://gerrit.cloudera.org:8080/17010 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I5c97b03475b0ffc7b387d7dfc17acc4b13858fb7 Gerrit-Change-Number: 17010 Gerrit-PatchSet: 8 Gerrit-Owner: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 17 Feb 2021 23:27:21 +0000 Gerrit-HasComments: Yes
