Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14953 )
Change subject: [tests] address flakiness in raft_consensus_election-itest ...................................................................... [tests] address flakiness in raft_consensus_election-itest Few test scenarios of the raft_consensus_election-itest suite involving churny elections were showing flakiness when run in slow mode with --stress_cpu_threads=16. The common root of the problem was failing writer test thread due to timeout. This patch addresses the issue, increasing Raft heartbeat interval from 1 to 2 milliseconds. With this change, the above mentioned tests become more stable, no longer failing due to the timeout error. I ran the raft_consensus_election-itest built in DEBUG mode multiple 1K batches to confirm that. Even with this patch, the above mentioned test scenarios sometimes fail due to the DCHECK_GE assert in PeerMessageQueue::CheckMonotonicTerms(). The latter issues will be addressed separately. Change-Id: If6f54643c9c066b31a74e1082260225e60324e4e Reviewed-on: http://gerrit.cloudera.org:8080/14953 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/consensus/consensus_queue.cc M src/kudu/consensus/consensus_queue.h M src/kudu/integration-tests/raft_consensus_election-itest.cc 3 files changed, 19 insertions(+), 20 deletions(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/14953 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: If6f54643c9c066b31a74e1082260225e60324e4e Gerrit-Change-Number: 14953 Gerrit-PatchSet: 5 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241)
