Adar Dembo has posted comments on this change.

Change subject: [java client] Fix an NPE in KuduException
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/4488/1/java/kudu-client/src/main/java/org/apache/kudu/client/KuduException.java
File java/kudu-client/src/main/java/org/apache/kudu/client/KuduException.java:

PS1, Line 83: // Message is allowed to be null, so if we let it go through 
we'll NPE building the Status
            :     // down the line.
Nit: second part of the sentence is self-evident. I'd just say "The message may 
be null." (since that is surprising).

Where do these null messages come from?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iceddc6931e8d3a8cb807657fc5c0804f7052e48f
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: Kudu Jenkins
Gerrit-HasComments: Yes

Reply via email to