Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-237 (part 1) - Support proper mutation encoding for reinserts ......................................................................
KUDU-237 (part 1) - Support proper mutation encoding for reinserts Currently reinserts are not properly encoded like the other mutations and instead refer to an in-memory row, meaning we can't flush them as they don't include the indirect data. This patch makes it so that reinserts are properly encoded, like other mutations but does not yet include the step of flushing them to disk, which will be done in a follow-up patch. Change-Id: I2b0a95e3f618bf20eb18ca4f4b03b9e2ce5704e8 Reviewed-on: http://gerrit.cloudera.org:8080/4791 Tested-by: Kudu Jenkins Reviewed-by: Todd Lipcon <t...@apache.org> --- M src/kudu/common/row_changelist-test.cc M src/kudu/common/row_changelist.cc M src/kudu/common/row_changelist.h M src/kudu/tablet/compaction.cc M src/kudu/tablet/memrowset-test.cc M src/kudu/tablet/memrowset.cc 6 files changed, 295 insertions(+), 238 deletions(-) Approvals: Todd Lipcon: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/4791 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2b0a95e3f618bf20eb18ca4f4b03b9e2ce5704e8 Gerrit-PatchSet: 18 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org>