Alexey Serbin has submitted this change and it was merged.

Change subject: [client-test] fixed flakiness in TestWriteTimeout
......................................................................


[client-test] fixed flakiness in TestWriteTimeout

In case of sanitizer builds, running the test along with other
concurrent activity could lead to errors like

ClientTest.TestWriteTimeout: client-test.cc:2130: Failure
Value of: error->status().ToString()
Expected: has substring "Write RPC to 127.0.0.1:"
  Actual: "Timed out: Failed to write batch of 1 ops to tablet \
    feb544f396a94a94bfb832b7785376e5 after 2 attempt(s): \
    Failed to write to server: (no server available): \
    Write(tablet: feb544f396a94a94bfb832b7785376e5, num_ops: 1, \
      num_attempts: 2) passed its deadline: \
    Timed out: connection negotiation to 127.0.0.1:33093 for RPC Write \
      timed out after 0.096s (ON_OUTBOUND_QUEUE)"

because it takes too long to negotiate a connection to the server.

This patch increases the operation timeout so now the test passes
for 1K+ runs without any errors.

I also increased session timeout for non-sanitizer builds from 100ms
to 200ms to have make the test more stable in other builds as well.

Change-Id: I2ced444038e721def7212dc8133318d5542c661f
Reviewed-on: http://gerrit.cloudera.org:8080/7006
Reviewed-by: Mike Percy <[email protected]>
Tested-by: Kudu Jenkins
---
M src/kudu/client/client-test.cc
1 file changed, 19 insertions(+), 11 deletions(-)

Approvals:
  Mike Percy: Looks good to me, approved
  Kudu Jenkins: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2ced444038e721def7212dc8133318d5542c661f
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <[email protected]>

Reply via email to