Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17559 )
Change subject: [test] make new scenario in security-itest more robust ...................................................................... [test] make new scenario in security-itest more robust This patch makes the newly introduced test scenario SecurityITest.TxnSmokeWithDifferentUserTypes more robust and stable. With awaiting for the first transaction to finalize, now there is no risk for aborting the second transaction in case of a race. Before this patch, if it took a long time to finalize aborting the former transaction, the deadlock prevention logic would automatically abort the latter one, failing the test scenario. Before this fix, 1/4 (one if four) of runs failed if running with --stress_cpu_threads=16. With this fix, I didn't see a failure in hundreds of runs. Change-Id: Ie99ad41a7a161f9046ba894c072eb92150f0e0e2 Reviewed-on: http://gerrit.cloudera.org:8080/17559 Tested-by: Kudu Jenkins Reviewed-by: Andrew Wong <[email protected]> --- M src/kudu/integration-tests/security-itest.cc 1 file changed, 14 insertions(+), 0 deletions(-) Approvals: Kudu Jenkins: Verified Andrew Wong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/17559 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ie99ad41a7a161f9046ba894c072eb92150f0e0e2 Gerrit-Change-Number: 17559 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
