Hello Will Berkeley, Mike Percy,

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

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

to review the following change.


Change subject: KUDU-2687: another attempt at fixing ITClient retries
......................................................................

KUDU-2687: another attempt at fixing ITClient retries

Perhaps they're broken because, when the test fails, we never join the
test threads? This patch addresses that by reworking the latch usage. It
also removes one of the latches; I don't think both were needed.

Change-Id: I31ec2d708d0ed41f25abcf09011967062c9a56c6
---
M java/kudu-client/src/test/java/org/apache/kudu/client/ITClient.java
1 file changed, 23 insertions(+), 17 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I31ec2d708d0ed41f25abcf09011967062c9a56c6
Gerrit-Change-Number: 12820
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Will Berkeley <[email protected]>

Reply via email to