Todd Lipcon has posted comments on this change.

Change subject: [python] - Allow to set a lower/upper bound on the scanner
......................................................................


Patch Set 1:

(2 comments)

would a tuple-based API be more pythonic here? Given that PKs are un-alterable, 
it seems like most people would want to specify as a tuple rather than the 
map-like interface provided by the PartialRow subclass.

http://gerrit.cloudera.org:8080/#/c/2030/1//COMMIT_MSG
Commit Message:

Line 11: XX
?


http://gerrit.cloudera.org:8080/#/c/2030/1/python/kudu/client.pyx
File python/kudu/client.pyx:

Line 1212:         bint applied
hrm, this isn't used?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie7943c913cec957015202ae0d8f8a4add0096fc0
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: Yes

Reply via email to