Alexey Serbin has posted comments on this change. Change subject: consensus: KUDU-2147. Unknown leader should not be treated as valid UUID ......................................................................
Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/8109/1/src/kudu/integration-tests/raft_config_change-itest.cc File src/kudu/integration-tests/raft_config_change-itest.cc: PS1, Line 56: The : // master should update its record of which replica is the leader after a new : // leader is elected. > correct - the last WaitForServersToAgree() would fail because the master wo Could you add a comment about this along with that last call to WaitForServersToAgree() ? PS1, Line 130: // Wait until the master re-adds the evicted replica and it is back up and : // running. : ASSERT_OK(WaitForServersToAgree(kTimeout, ts_map_, tablet_id, 1)); > It wouldn't buy us much. This function retries. Yes, but I thought the cluster configuration should be brought back, so the removed follower is in the cluster again, if you want to start with the same cluster. -- To view, visit http://gerrit.cloudera.org:8080/8109 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie882d05fc58e55836edc0235d14974e65125df6c Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
