Hello Tidy Bot, Alexey Serbin, Zoltan Martonka, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/20546
to look at the new patch set (#2).
Change subject: [compaction] Skip memory allocation for ancient undo deltas
......................................................................
[compaction] Skip memory allocation for ancient undo deltas
Currently, while applying REDO mutations to base row and create
corresponding UNDO deltas for each REDO mutation, we don't check
whether any mutation is ancient that is anyway going to be ignored
and removed from list of UNDO deltas in later stage of processing.
This change checks each REDO mutation beforehand and doesn't allocate
any memory if found ancient. This avoids unnecessary memory usage.
Change-Id: Ibb41636a3ac063478fe181560d4ec85dadeb0ef3
---
M src/kudu/tablet/compaction.cc
M src/kudu/tablet/compaction.h
M src/kudu/tablet/delta_compaction.cc
M src/kudu/tablet/tablet-test-base.h
M src/kudu/tablet/tablet_history_gc-test.cc
5 files changed, 121 insertions(+), 16 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/46/20546/2
--
To view, visit http://gerrit.cloudera.org:8080/20546
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ibb41636a3ac063478fe181560d4ec85dadeb0ef3
Gerrit-Change-Number: 20546
Gerrit-PatchSet: 2
Gerrit-Owner: Ashwani Raina <ara...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ara...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Zoltan Martonka <zmarto...@cloudera.com>
