Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17562 )
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]> (cherry picked from commit 598304262439fed0819169d6b1a6bc56020a5eea) Reviewed-on: http://gerrit.cloudera.org:8080/17562 Tested-by: Andrew Wong <[email protected]> Reviewed-by: Bankim Bhavsar <[email protected]> --- M src/kudu/integration-tests/security-itest.cc 1 file changed, 14 insertions(+), 0 deletions(-) Approvals: Andrew Wong: Looks good to me, approved; Verified Bankim Bhavsar: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/17562 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.15.x Gerrit-MessageType: merged Gerrit-Change-Id: Ie99ad41a7a161f9046ba894c072eb92150f0e0e2 Gerrit-Change-Number: 17562 Gerrit-PatchSet: 2 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)
