Adar Dembo has submitted this change and it was merged. Change subject: fs: add BlockManager::GetAllBlockIds() ......................................................................
fs: add BlockManager::GetAllBlockIds() This method will be used in a poor man's data block GC. It's simple enough to implement in the LBM, where all blocks are known, but more complicated in the FBM. Change-Id: I20e8ccf6e8a2deba88fcf5598fb404a1186b8262 Reviewed-on: http://gerrit.cloudera.org:8080/6360 Tested-by: Adar Dembo <[email protected]> Reviewed-by: David Ribeiro Alves <[email protected]> --- M src/kudu/fs/block_id.h 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/log_block_manager.cc M src/kudu/fs/log_block_manager.h M src/kudu/tablet/compaction-test.cc M src/kudu/tablet/rowset_metadata.cc M src/kudu/tablet/tablet_metadata.h M src/kudu/tserver/tablet_copy_source_session.h M src/kudu/util/env.h 13 files changed, 161 insertions(+), 22 deletions(-) Approvals: David Ribeiro Alves: Looks good to me, approved Adar Dembo: Verified -- To view, visit http://gerrit.cloudera.org:8080/6360 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I20e8ccf6e8a2deba88fcf5598fb404a1186b8262 Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Dinesh Bhat <[email protected]> Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]>
