Hello Kudu Jenkins, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/12276
to look at the new patch set (#4).
Change subject: [java] deflake RYW tests in TestKuduClient
......................................................................
[java] deflake RYW tests in TestKuduClient
RYW tests in TestKuduClient is flaky which fails with AssertionError
when verifying the chosen snapshot timestamp returned from the server
is larger than the previous propagated timestamp before the scan.
It turns out to be a test only issue, which due to incorrect structure
of the test. It aims to test READ_YOUR_WRITE mode for multiple clients,
while the same client was being used. The distinction is important,
because we want to ensure a single client can still preserve
read-your-writes and read-your-reads session guarantees even there are
concurrent reads/writes performed by other clients in READ_YOUR_WRITE
scan mode.
Without the fix, 192/1000 runs of TestKuduClient failed with this error.
With the fix, 0/1000 runs of TestKuduClient failed.
Change-Id: I951abb9197f7e6b6a4c70cdf89948206840ddeda
---
M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduScanner.java
M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClient.java
2 files changed, 15 insertions(+), 8 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/76/12276/4
--
To view, visit http://gerrit.cloudera.org:8080/12276
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I951abb9197f7e6b6a4c70cdf89948206840ddeda
Gerrit-Change-Number: 12276
Gerrit-PatchSet: 4
Gerrit-Owner: Hao Hao <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
