Hello Alexey Serbin, Dan Burkert, Jean-Daniel Cryans, Kudu Jenkins, Todd Lipcon,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8382
to look at the new patch set (#2).
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.1508955740.12232
Change-Id: I90ffcd01e7f99f3090fa118092fc303e06fb92dc
---
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, 27 insertions(+), 24 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/82/8382/2
--
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: newpatchset
Gerrit-Change-Id: I90ffcd01e7f99f3090fa118092fc303e06fb92dc
Gerrit-Change-Number: 8382
Gerrit-PatchSet: 2
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]>
