Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17691 )
Change subject: [master] make automatic AddMaster resilient to leadership changes ...................................................................... Patch Set 2: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/17691/2//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/17691/2//COMMIT_MSG@21 PS2, Line 21: This patch also addresses some leftover feedback from : 7e66534d0e62fb850bf300d52da4a0a76889f4b8 regarding verification of : masters in the presence of DNS failures. I didn't add a new test for : this because it turned out to not change the behavior of things -- the : new master would still fail upon attempting to resolve replicas. Thank you for addressing this! -- To view, visit http://gerrit.cloudera.org:8080/17691 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie38453c6fc41ce98c59c010902e2d9fe9db62dee Gerrit-Change-Number: 17691 Gerrit-PatchSet: 2 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 20 Jul 2021 17:48:09 +0000 Gerrit-HasComments: Yes
