Jean-Daniel Cryans has posted comments on this change.

Change subject: [java client] Fix GetTableLocations error handling
......................................................................


Patch Set 3:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/3295/3/java/kudu-client/src/main/java/org/kududb/client/ListTabletsResponse.java
File java/kudu-client/src/main/java/org/kududb/client/ListTabletsResponse.java:

Line 24: @InterfaceAudience.Public
I would mark it private since we're not exposing anyway of using it.


PS3, Line 39: sL
nit: Tablet*


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I79d246a83e9f0071063a23fb9a8d4b4c0220e507
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Binglin Chang <decst...@gmail.com>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: Yes

Reply via email to