Hello Adar Dembo,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/12264
to review the following change.
Change subject: revert test changes from KUDU-2236
......................................................................
revert test changes from KUDU-2236
A couple of changes (d78b2727d1246069b2006ee652c3ff9a2005601c,
I7aafdc0eca00e743048ecc099dcb3241ce7ac8ad) for KUDU-2236 went in to make
testCloseShortlyAfterOpen less flaky, but failed to actually fix the
underlying logging regression.
Commit ead756844ce9ada904fcc3666df25692f63e76b8 fixed the regression, so
this patch restores the test to its former coverage. I looped the test
and it passed 200/200 runs, compared to the failure rate of 20-50%
reported in the Jira.
The above commit added a similar test, but given the tests test
different things, I've left both in.
Change-Id: Ia93a454b06a40738152cc000d55caa197b64d461
---
M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClient.java
1 file changed, 4 insertions(+), 3 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/64/12264/1
--
To view, visit http://gerrit.cloudera.org:8080/12264
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia93a454b06a40738152cc000d55caa197b64d461
Gerrit-Change-Number: 12264
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
