David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/4928
Change subject: Make RowChangeListDecoder::RemoveColumnIdsFromChangeList() take a RowChangeListEncoder as an out param ...................................................................... Make RowChangeListDecoder::RemoveColumnIdsFromChangeList() take a RowChangeListEncoder as an out param All the other methods in this class, of this genre, take one but this one was taking the raw faststring buffer insted. Though it did have the advantage of not forcing the caller to clear() the buffer everytime it was weirdly out of place and might introduce subtle bugs with new semantics for RowChangeListEncoder. Change-Id: I6584b22662b58227a80eaebb7d89199d3ea2498a --- M src/kudu/common/row_changelist.cc M src/kudu/common/row_changelist.h M src/kudu/tablet/memrowset.cc 3 files changed, 11 insertions(+), 11 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/28/4928/1 -- To view, visit http://gerrit.cloudera.org:8080/4928 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I6584b22662b58227a80eaebb7d89199d3ea2498a Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: David Ribeiro Alves <dral...@apache.org>