Adar Dembo has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11529 )

Change subject: [spark] Cleanup hasNext logic
......................................................................


Patch Set 1:

If you think that's bad, take a look at 
https://github.com/apache/kudu/blob/master/java/kudu-mapreduce/src/main/java/org/apache/kudu/mapreduce/KuduTableInputFormat.java#L428.
 Maybe that's something you can fix too?


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I340af13a695b3a8489e03018542b8ad7db614758
Gerrit-Change-Number: 11529
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Comment-Date: Wed, 26 Sep 2018 22:04:23 +0000
Gerrit-HasComments: No

Reply via email to