Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] Few ITClient improvements ......................................................................
[java client] Few ITClient improvements ITClient has been flaky for a while now, mostly due to the "Row count regressed" issue. I fixed it by using snapshot timestamps, which made me refactor how we build scanners, which made me add a new counting method in BaseKuduTest. I continued running the test and saw other issues. Some unchecked errors were not killing the test, so I added an UncaughtExceptionHandler. I also saw invalid scanner sequence ID errors that are normal due to how this test runs that were killing the test. Finally, I converted some plain Exceptions into KuduExceptions which gave us access to their Status. Change-Id: I3b5ddca26b66e9fc1f737aaacf98df340f0b9024 Reviewed-on: http://gerrit.cloudera.org:8080/4489 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <a...@cloudera.com> --- M java/kudu-client/src/test/java/org/apache/kudu/client/BaseKuduTest.java M java/kudu-client/src/test/java/org/apache/kudu/client/ITClient.java 2 files changed, 68 insertions(+), 26 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/4489 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3b5ddca26b66e9fc1f737aaacf98df340f0b9024 Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Dan Burkert <d...@cloudera.com> Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Kudu Jenkins