You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Todd Lipcon (Code Review)" <ge...@cloudera.org> on 2016/03/09 23:28:39 UTC

[kudu-CR] client-test: try to de-flake TestLastErrorEmbeddedInScanTimeoutStatus

Hello Adar Dembo,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/2506

to review the following change.

Change subject: client-test: try to de-flake TestLastErrorEmbeddedInScanTimeoutStatus
......................................................................

client-test: try to de-flake TestLastErrorEmbeddedInScanTimeoutStatus

This test seems to be flaky on the test dashboard, though I couldn't
reproduce it on dist-test (perhaps due to different slave hardware).

Regardless, it seems like there's a race -- we inject 1 second of
latency into the tablet server startup, and then set a 1-second
timeout on our scan. So, maybe our scan can sometimes succeed within
the target time in some cases.

This bumps the latency to 5 seconds to try to make the intended timeout
more of a sure thing.

Change-Id: I63c3e31a737b11ad3c9104d2f77111d17e9a51b7
---
M src/kudu/client/client-test.cc
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/06/2506/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2506
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I63c3e31a737b11ad3c9104d2f77111d17e9a51b7
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>