Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8868 )
Change subject: consensus: Fix NON_VOTER ack-counting bug ...................................................................... consensus: Fix NON_VOTER ack-counting bug This patch fixes an issue where we were not differentiating between replicating to voters and non-voters. This enables the test written by Alexey and also makes some changes to it. The test fails without this patch and passes with the patch. Tests added: * Added a unit test in consensus_queue-test * Updated and enabled the system test in raft_consensus_nonvoter-itest Change-Id: I13143e9bb4b76af3fd6dada28fcec05b27d24476 Reviewed-on: http://gerrit.cloudera.org:8080/8868 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/consensus/consensus-test-util.h M src/kudu/consensus/consensus_peers-test.cc M src/kudu/consensus/consensus_peers.cc M src/kudu/consensus/consensus_queue-test.cc M src/kudu/consensus/consensus_queue.cc M src/kudu/consensus/consensus_queue.h M src/kudu/consensus/raft_consensus.cc M src/kudu/integration-tests/raft_consensus_nonvoter-itest.cc 8 files changed, 233 insertions(+), 88 deletions(-) Approvals: Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- 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: merged Gerrit-Change-Id: I13143e9bb4b76af3fd6dada28fcec05b27d24476 Gerrit-Change-Number: 8868 Gerrit-PatchSet: 7 Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Tidy Bot
