Hello Alexey Serbin, Kudu Jenkins, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14980
to look at the new patch set (#2).
Change subject: test: deflake RaftConsensusNumLeadersMetricTest
TestNumLeadersMetric
......................................................................
test: deflake RaftConsensusNumLeadersMetricTest TestNumLeadersMetric
The test previously wouldn't wait for all tablets to be up and running
on every tserver before doing stuff. This led to errors when attempting
to start elections, since the StartElection RPC would be eaten and never
retried, leaving some tablets with no leader.
The fix is to wait for every tserver to have all running tablets.
Before this fix, the test would fail in TSAN mode 6/1000 times. With it,
it passed 1000/1000.
Change-Id: I4ef4651e4562ac274982773ebb8c46e5137e38b5
---
M src/kudu/integration-tests/raft_consensus_election-itest.cc
1 file changed, 4 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/80/14980/2
--
To view, visit http://gerrit.cloudera.org:8080/14980
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I4ef4651e4562ac274982773ebb8c46e5137e38b5
Gerrit-Change-Number: 14980
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
