Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8325 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/8325 Reviewed-by: Adar Dembo <a...@cloudera.com> Tested-by: Adar Dembo <a...@cloudera.com> --- 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(-) Approvals: Adar Dembo: Looks good to me, approved; Verified -- 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: merged Gerrit-Change-Id: I887e20643ac8e4e728f8f91450d2d5059cb33d20 Gerrit-Change-Number: 8325 Gerrit-PatchSet: 4 Gerrit-Owner: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>
