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 (#2). 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(+), 15 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/25/8325/2 -- 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: 2 Gerrit-Owner: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins