Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/10887 )

Change subject: [raft_consensus-itest] fix TestElectionMetrics flake
......................................................................

[raft_consensus-itest] fix TestElectionMetrics flake

This patch fixes flakiness in the RaftConsensusITest.TestElectionMetrics
scenario.  The original TestElectionMetrics scenario was split into
two parts TestElectionMetricsPart[1,2] to separate tests that assume
no leader election happen and depend on the leader failure detection
mechanism, correspondingly.

Prior to this patch, the TestElectionMetrics scenario could fail if a
leader election happened inadvertently.

Before (1 out of 12 failed):
  http://dist-test.cloudera.org/job?job_id=aserbin.1530681412.7635

After  (not a single failure in 1K run):
  http://dist-test.cloudera.org/job?job_id=aserbin.1530918429.120803

Change-Id: I073c9989a6d5d5dc1eb104120a89d38cfce2ac6e
Reviewed-on: http://gerrit.cloudera.org:8080/10887
Tested-by: Alexey Serbin <aser...@cloudera.com>
Reviewed-by: Will Berkeley <wdberke...@gmail.com>
Reviewed-by: Attila Bukor <abu...@cloudera.com>
---
M src/kudu/consensus/raft_consensus.cc
M src/kudu/integration-tests/raft_consensus-itest.cc
2 files changed, 121 insertions(+), 27 deletions(-)

Approvals:
  Alexey Serbin: Verified
  Will Berkeley: Looks good to me, approved
  Attila Bukor: Looks good to me, but someone else must approve

--
To view, visit http://gerrit.cloudera.org:8080/10887
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I073c9989a6d5d5dc1eb104120a89d38cfce2ac6e
Gerrit-Change-Number: 10887
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <abu...@cloudera.com>
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>

Reply via email to