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