Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8868 )
Change subject: consensus: Fix NON_VOTER ack-counting bug ...................................................................... Patch Set 2: Overall looks good, maybe just those nits from PS1. Also, do you think it's worth separating the patch which requires the presence of the member type in a element of the the consensus queue? -- To view, visit http://gerrit.cloudera.org:8080/8868 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I13143e9bb4b76af3fd6dada28fcec05b27d24476 Gerrit-Change-Number: 8868 Gerrit-PatchSet: 2 Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Comment-Date: Tue, 19 Dec 2017 01:55:36 +0000 Gerrit-HasComments: No
