Hello Dinesh Bhat, Mike Percy,

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

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

to review the following change.

Change subject: Fix flakiness in ToolTest.TestRemoteReplicaCopy
......................................................................

Fix flakiness in ToolTest.TestRemoteReplicaCopy

This test created a table, and then immediately asserted on the error
message returned when trying to copy a tablet on top of an existing
tablet. In fact, the destination tablet might still be in the 'creating'
transition, in which case the copy attempt failed with an unexpected
error message.

This patch fixes the issue by waiting for the RUNNING state first.

Tested a loop 500x with 4 stress threads in a RELEASE build (where this
seemed to be most flaky according to the dashboard).

Change-Id: I740f6b469561a17792a315caf4a10c2627b55149
---
M src/kudu/tools/kudu-tool-test.cc
1 file changed, 5 insertions(+), 1 deletion(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I740f6b469561a17792a315caf4a10c2627b55149
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Dinesh Bhat <[email protected]>
Gerrit-Reviewer: Mike Percy <[email protected]>

Reply via email to