Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11929 )
Change subject: test: deflake TestRestartWithPendingCommitFromFailedOp ...................................................................... test: deflake TestRestartWithPendingCommitFromFailedOp The test would fail to create the tablet due to the injected failure, when it should have really waited until the tablet was bootstrapped before injecting errors. The test failed 6/1000 times with stress and with a seed of -922249903. With the fix, 1000/1000 passed. This isn't the only source of flakiness in ts_recovery-itest, but it's something. Change-Id: Ic76b5f120aee7327f6cee7324b862bb89c5662ab Reviewed-on: http://gerrit.cloudera.org:8080/11929 Tested-by: Andrew Wong <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/integration-tests/ts_recovery-itest.cc 1 file changed, 2 insertions(+), 3 deletions(-) Approvals: Andrew Wong: Verified Alexey Serbin: Looks good to me, approved Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/11929 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ic76b5f120aee7327f6cee7324b862bb89c5662ab Gerrit-Change-Number: 11929 Gerrit-PatchSet: 2 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) Gerrit-Reviewer: Todd Lipcon <[email protected]>
