Hello Tidy Bot, Alexey Serbin, Zoltan Martonka, Kudu Jenkins, Wang Xixu,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/20546
to look at the new patch set (#5).
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, compaction code
doesn'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, 206 insertions(+), 16 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/46/20546/5
--
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: 5
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: Wang Xixu <1450306...@qq.com>
Gerrit-Reviewer: Zoltan Martonka <zmarto...@cloudera.com>
