David Ribeiro Alves has posted comments on this change.

Change subject: [java client] Reinstate KUDU-1364's behavior, fix NPE
......................................................................


Patch Set 1: Code-Review+2

thanks for fixing this.
yeah, it'd be good to have directed tests for this stuff, though it would take 
quite some time to assemble a fw to do it. the inability to write those is 
likely the reason why the other patches were flawed.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8e0ed23fbf4c655037b77173a187c3fa11de4f63
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: David Ribeiro Alves <dral...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: No

Reply via email to