Hello Kudu Jenkins, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8325
to look at the new patch set (#3).
Change subject: Integrate BlockCreationTransaction into MajorDeltaCompaction
......................................................................
Integrate BlockCreationTransaction into MajorDeltaCompaction
Previous commit 59ca360bf integrated BlockCreationTransaction with
RowSet compaction, so that only a single transaction is used to finalize
underlying blocks. However in major delta compaction, MultiColumnWriter
and DeltaFileWriter use separate creation transactions, which does not
take full advantage of BlockCreationTransaction.
This patch includes a minor improvement to incorporate
BlockCreationTransaction with MajorDeltaCompaction to use a single
transaction for a RowSet's major delta compaction.
Change-Id: I887e20643ac8e4e728f8f91450d2d5059cb33d20
---
M src/kudu/tablet/delta_compaction.cc
M src/kudu/tablet/multi_column_writer.cc
M src/kudu/tablet/multi_column_writer.h
3 files changed, 8 insertions(+), 16 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/25/8325/3
--
To view, visit http://gerrit.cloudera.org:8080/8325
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I887e20643ac8e4e728f8f91450d2d5059cb33d20
Gerrit-Change-Number: 8325
Gerrit-PatchSet: 3
Gerrit-Owner: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
