Hao Hao has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12276 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/12276 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <a...@cloudera.com> --- 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(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved -- 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: merged Gerrit-Change-Id: I951abb9197f7e6b6a4c70cdf89948206840ddeda Gerrit-Change-Number: 12276 Gerrit-PatchSet: 5 Gerrit-Owner: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120)