Hello Alexey Serbin, Adar Dembo,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/14980
to review the following change.
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, 5 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/80/14980/1
--
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: newchange
Gerrit-Change-Id: I4ef4651e4562ac274982773ebb8c46e5137e38b5
Gerrit-Change-Number: 14980
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
