Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8138 )
Change subject: WIP [consensus] adding/removing NON_VOTER members ...................................................................... Patch Set 7: > (8 comments) > > This is looking pretty good although I have some comments. I don't > think we need to add client tool support to this patch before > merging it, since it simply adds infrastructure and features that > are not possible to activate without extraordinary effort. Thank you for the review. Yes, agree that client tool support should be a separate patch. Actually, I already created a separate patch to introduce replica selector: https://gerrit.cloudera.org/#/c/8161/ I'm basing my patch for kudu CLI tool changes based on that. -- To view, visit http://gerrit.cloudera.org:8080/8138 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2662d45ad9bb6a4bf325d4202c2ee619ffad02b7 Gerrit-Change-Number: 8138 Gerrit-PatchSet: 7 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: David Ribeiro Alves <davidral...@gmail.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-Comment-Date: Thu, 05 Oct 2017 18:55:09 +0000 Gerrit-HasComments: No