Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14980 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/14980 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/integration-tests/raft_consensus_election-itest.cc 1 file changed, 4 insertions(+), 2 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved Kudu Jenkins: Verified -- 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: merged Gerrit-Change-Id: I4ef4651e4562ac274982773ebb8c46e5137e38b5 Gerrit-Change-Number: 14980 Gerrit-PatchSet: 4 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
