Adar Dembo has posted comments on this change.

Change subject: Release notes for the Java client in 1.1
......................................................................


Patch Set 1:

(5 comments)

http://gerrit.cloudera.org:8080/#/c/5002/1/docs/release_notes.adoc
File docs/release_notes.adoc:

PS1, Line 50: troubleshooting
troubleshoot


PS1, Line 54:  will not
"are not" (twice in this sentence)


PS1, Line 59: currently
            :   available
Replace with "existing"


PS1, Line 86: are not using the default
            :   timeouts anymore to wait for the requests to complete. 
This part may be too much detail. Maybe it's enough to just say that before the 
sync API could throw two kinds of exceptions, now just one?


Line 91: * The Java client's handling of errors in `KuduSession` was modified 
so that subclasses of
Shouldn't this specifically refer to Flush() or Apply()? Not clear where these 
exceptions were being thrown from, or under what circumstances.


-- 
To view, visit http://gerrit.cloudera.org:8080/5002
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idf04d1db4769e19e6421628bbe980cb4c058c62c
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Jean-Daniel Cryans <jdcry...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: John Russell <jruss...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: Yes

Reply via email to