Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8382 )
Change subject: [java client] improve AsyncKuduScanner logging ...................................................................... [java client] improve AsyncKuduScanner logging As filed in KUDU-2199, in a jenkin's job build, ITClient test failed even with fault tolerant scanner. The test log demonstrates that the issue may cause by re-scan of scanned data. However, loop ITClient 10000 times and did not discover any issues: http://dist-test.cloudera.org/job?job_id=hao.hao.1508869382.32617 This patch improves test coverage of ITClient so that error will be reported if row count of each scan ever unexpectedly decrease. It also improves logging of AsyncKuduScanner to give more information for diagnosis if the same issue occur again. Another 2000 runs of ITClient with current patch passed: http://dist-test.cloudera.org/job?job_id=hao.hao.1509045968.14317 Change-Id: I90ffcd01e7f99f3090fa118092fc303e06fb92dc Reviewed-on: http://gerrit.cloudera.org:8080/8382 Reviewed-by: Dan Burkert <[email protected]> Tested-by: Kudu Jenkins --- M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduScanner.java R java/kudu-client/src/main/java/org/apache/kudu/client/FaultTolerantScannerExpiredException.java M java/kudu-client/src/test/java/org/apache/kudu/client/ITClient.java 3 files changed, 22 insertions(+), 19 deletions(-) Approvals: Dan Burkert: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8382 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I90ffcd01e7f99f3090fa118092fc303e06fb92dc Gerrit-Change-Number: 8382 Gerrit-PatchSet: 6 Gerrit-Owner: Hao Hao <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
