Jordan Birdsell has posted comments on this change.

Change subject: [python] added KuduPartialRow::SetNoCopy{Binary,String}()
......................................................................


Patch Set 2:

Ah, i see, so in that case are you thinking we should use the nocopy routine to 
continue as was done before? If so you'll want to also update the PartialRow 
stuff, otherwise this code does nothing.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: If73894c3037f1158cf7b9d3aafecd228a0204bbd
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Jordan Birdsell <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: No

Reply via email to