Advertising

Patch Set 2:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/3330/2/java/kudu-client/src/main/java/org/kududb/client/AsyncKuduClient.java
File java/kudu-client/src/main/java/org/kududb/client/AsyncKuduClient.java:
Line 1004: message = "Time out: ";
I wonder if it should be even clearer than that. The reason we got "Too many
attempts" before is because were were checking wouldSleepingTimeout() and it
was returning true. So we early time out the RPC to respect the configured
timeout.
So now we get "Time out", except that the log will still show something like
"DeadlineTracker(timeout=30000, elapsed=28824)".
--
To view, visit http://gerrit.cloudera.org:8080/3330
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I9c5676ab5bd05170505ef0313b919c5475ae6b37
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <d...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: Yes