Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8219 to look at the new patch set (#2). Change subject: KUDU-2055 [part 4]: refactor BM to remove BlockManager::DeleteBlock ...................................................................... KUDU-2055 [part 4]: refactor BM to remove BlockManager::DeleteBlock This patch refactors block manager to use fs::BlockDeletionTransaction instead of BlockManager::DeleteBlock for block deletions, in order to simplify the interfaces of block manager and have a uniform path for block deletions. Change-Id: Ied2ab44f33bce29706f8f41acc3d468582edad6e --- M src/kudu/fs/block_manager-stress-test.cc M src/kudu/fs/block_manager-test.cc M src/kudu/fs/block_manager.h M src/kudu/fs/file_block_manager.cc M src/kudu/fs/file_block_manager.h M src/kudu/fs/fs_manager.cc M src/kudu/fs/log_block_manager-test.cc M src/kudu/fs/log_block_manager.h M src/kudu/tablet/tablet_metadata.cc M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/tool_action_fs.cc M src/kudu/tserver/tablet_copy_source_session-test.cc 12 files changed, 132 insertions(+), 59 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/19/8219/2 -- To view, visit http://gerrit.cloudera.org:8080/8219 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ied2ab44f33bce29706f8f41acc3d468582edad6e Gerrit-Change-Number: 8219 Gerrit-PatchSet: 2 Gerrit-Owner: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Kudu Jenkins