Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/16539 )

Change subject: [tests] fix flake in TestTabletServerProxyCallErrors
......................................................................

[tests] fix flake in TestTabletServerProxyCallErrors

This patch fixes a flake in the TestTabletServerProxyCallErrors
scenario of the TxnStatusTabletManagementTest. Before this patch,
it was failing in about 1 out of 16 runs when running with
--stress_cpu_threads=16 (DEBUG build) with errors like below:

  src/kudu/integration-tests/ts_tablet_manager-itest.cc:1325: Failure
  Value of: s.IsInvalidArgument()
    Actual: false
  Expected: true
  Illegal state: Tablet not RUNNING: BOOTSTRAPPING
  Google Test trace:
  src/kudu/integration-tests/ts_tablet_manager-itest.cc:1322: error {
    code: TABLET_NOT_RUNNING
    status {
      code: ILLEGAL_STATE
      message: "Tablet not RUNNING: BOOTSTRAPPING"
    }
  }

After this patch, none of 300+ runs of the scenario failed when running
with the same flags.

Change-Id: I64b9f1eeb6bacf684a15ef84cddacadb43ac43fe
Reviewed-on: http://gerrit.cloudera.org:8080/16539
Tested-by: Alexey Serbin <[email protected]>
Reviewed-by: Andrew Wong <[email protected]>
---
M src/kudu/integration-tests/ts_tablet_manager-itest.cc
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  Alexey Serbin: Verified
  Andrew Wong: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/16539
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I64b9f1eeb6bacf684a15ef84cddacadb43ac43fe
Gerrit-Change-Number: 16539
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to