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>