Todd Lipcon has submitted this change and it was merged. Change subject: Fix flakiness of RaftConsensusITest.TestLogIOErrorIsFatal ......................................................................
Fix flakiness of RaftConsensusITest.TestLogIOErrorIsFatal This test was flaky because it was assuming that the order of entries in the TServersDetail map matched the ordering of tablet servers in the minicluster's tablet server vector. This is not always the case, so the assertions waiting for nodes to crash sometimes timed out (since they were waiting on the wrong nodes). Change-Id: If62911f9517b60d82fac97bacd2ac01584be442f Reviewed-on: http://gerrit.cloudera.org:8080/3493 Reviewed-by: Adar Dembo <a...@cloudera.com> Tested-by: Kudu Jenkins --- M src/kudu/integration-tests/raft_consensus-itest.cc 1 file changed, 12 insertions(+), 8 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/3493 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: If62911f9517b60d82fac97bacd2ac01584be442f Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Dan Burkert <d...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org>