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 <[email protected]> --- 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 <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <[email protected]>
