Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8219 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/8219 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- 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.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 13 files changed, 140 insertions(+), 62 deletions(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved -- 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: merged Gerrit-Change-Id: Ied2ab44f33bce29706f8f41acc3d468582edad6e Gerrit-Change-Number: 8219 Gerrit-PatchSet: 5 Gerrit-Owner: Hao Hao <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins
