Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/4055
Change subject: IMPALA-3996: Migrate to updated Kudu insert string API ...................................................................... IMPALA-3996: Migrate to updated Kudu insert string API In Kudu 0.10 the Insert API changed for Strings and Binary values to avoid ambiguity around memory copying. They changed SetString() to copy the value given to it to avoid misuse, and added a SetStringNoCopy() with the old behavior. We upgraded to 0.10 which has some perf impact but is still correct. This changes the code to use the NoCopy call which was the behavior we had previously. See Kudu commit: https://github.com/apache/kudu/commit/48766a4ce17d422ced9a6ec78c9a9969ac44d8c9 Change-Id: I910c24724d0bc887b2d4a3e62ecdf72420a76f6f --- M be/src/exec/kudu-table-sink.cc 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/55/4055/1 -- To view, visit http://gerrit.cloudera.org:8080/4055 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I910c24724d0bc887b2d4a3e62ecdf72420a76f6f Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Matthew Jacobs <[email protected]>
