Adar Dembo has posted comments on this change.

Change subject: KUDU-1267: Calling getString() on RowResult for wrong type gave 
weird exception
......................................................................


Patch Set 2:

Thanks for making some changes. There were several suggestions you missed; I 
flagged them for you. Also, you missed the larger suggestion I made the first 
time around: "your editor appears to have made many indentation changes to 
RowResult.java. AFAICT they don't make the file any more or less compliant with 
a coding style, so could you please revert them? They're just unnecessary 
noise.".

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idc8c6e5e382ca9f6280072f59acf4257fd13fc6b
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Ted Malaska <ted.mala...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: No

Reply via email to