David Ribeiro Alves has posted comments on this change.

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


Patch Set 1:

(2 comments)

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

Line 11: XX
> ?
ctrl-X, ctrl-X ctrl-X dammit! :) (nano's exit shortcut)


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?
yes it is (see L 1233 or L 1270). I basically made the old WriteOperation be 
just a partial row (since it really only had 'applied' that was write only) and 
created WriteOperation (from which insert/update/delete already inherited) to 
hold 'applied'.


-- 
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: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: Yes

Reply via email to