Jordan Birdsell has posted comments on this change. Change subject: [python] added KuduPartialRow::SetNoCopy{Binary,String}() ......................................................................
Patch Set 2: Was this more for parity's sake or did you need this exposed? The libkudu_client module really just exposes the external signatures to python so if you want this to be exposed in the client it would need to be implemented in the PartialRow class in client.pyx. However, given the way python handles the semantics for PartialRows (schema.new_row({'foo': 'bar'} or alternatively row['foo'] = 'bar' -- 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 <aser...@cloudera.com> Gerrit-Reviewer: Jordan Birdsell <jtbirds...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-HasComments: No