Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9627 )
Change subject: KUDU-2342. consensus: use tighter bound for non-voter promotion ...................................................................... KUDU-2342. consensus: use tighter bound for non-voter promotion This patch replaces the NON_VOTER promotion heuristic based on the --consensus_promotion_max_wal_entries_behind gflag with a heuristic that roughly matches the promotion criteria outlined in Diego Ongaro's dissertation. The approach is that we track the operation count of the last batch successfully sent from leader to peer. If the peer is within that number of ops of the commit index, the peer is eligible for promotion. The aforementioned gflag has been removed. This patch passes existing tests and did well on manual testing. I will follow up on this patch with an automated test. Change-Id: Iff517f01d6dc25eb15d01593dd57b7dc0dd25956 Reviewed-on: http://gerrit.cloudera.org:8080/9627 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/consensus/consensus_queue.cc 1 file changed, 21 insertions(+), 14 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/9627 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iff517f01d6dc25eb15d01593dd57b7dc0dd25956 Gerrit-Change-Number: 9627 Gerrit-PatchSet: 5 Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]>
