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? We talked offline and the decision was not to separate those parts of the patch, since the fix itself depends on the presence of the correct member type for a peer in the 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 02:20:04 +0000 Gerrit-HasComments: No
