[kudu] 01/02: [test] fix flake in TestConcurrentRepeatedCommitCalls

[alexey]

This is an automated email from the ASF dual-hosted git repository.

alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git

commit 67018be8ba27480b050c11504df8a732f6a52daf
Author: Alexey Serbin <ale...@apache.org>
AuthorDate: Mon Feb 15 23:27:15 2021 -0800

    [test] fix flake in TestConcurrentRepeatedCommitCalls
    
    Before this patch, the TestConcurrentRepeatedCommitCalls scenario
    of the TxnCommitITest sometimes failed with an error like below:
    
      Bad status: Illegal state: Failed to write to server: 
206a8320b5ea4e4c80360fd3de898ab5 (127.0.93.65:36189): transaction ID 0 is not 
open: state: COMMITTED user: "slave"
    
    Before this patch, I saw such an error in about 1/8 of all DEBUG build
    runs if specifying --stress_cpu_threads=16.  After this patch, I saw
    none after running the scenario with the same flags hundreds of times.
    
    Change-Id: I39118b2b33503a18b77e4e4106979f141b5e5486
    Reviewed-on: http://gerrit.cloudera.org:8080/17069
    Tested-by: Alexey Serbin <aser...@cloudera.com>
    Reviewed-by: Andrew Wong <aw...@cloudera.com>
---
 src/kudu/integration-tests/txn_commit-itest.cc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/kudu/integration-tests/txn_commit-itest.cc 
b/src/kudu/integration-tests/txn_commit-itest.cc
index cf3bf8f..920d47e 100644
--- a/src/kudu/integration-tests/txn_commit-itest.cc
+++ b/src/kudu/integration-tests/txn_commit-itest.cc
@@ -683,6 +683,7 @@ TEST_F(TxnCommitITest, TestConcurrentCommitCalls) {
 // Test that committing the same transaction concurrently doesn't lead to any
 // issues.
 TEST_F(TxnCommitITest, TestConcurrentRepeatedCommitCalls) {
+  FLAGS_txn_status_manager_inject_latency_finalize_commit_ms = 1000;
   shared_ptr<KuduTransaction> txn;
   shared_ptr<KuduSession> txn_session;
   ASSERT_OK(BeginTransaction(participant_ids_, &txn, &txn_session));