Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14862 )
Change subject: fs: rename PathInstance-related things ...................................................................... fs: rename PathInstance-related things I'm expecting the directory management code to be reused for WAL directories, and I've sometimes found it mildly annoying that we have different nomenclature for path instances, directories, and block managers in the same area (mostly in the PathInstances). So thought I'd rename some things now to make some things more straight-forward: * PathInstanceMetadataFile -> DirInstanceMetadataFile * PathInstanceMetadataPB -> DirInstanceMetadataPB * PathSetPB -> DirSetPB * block_manager_type -> dir_type (in the context of instance files only) * block_manager_util -> dir_util Along the way, I modernized our usage of the DataDir threadpool when calling DataDirManager::Open(). Change-Id: I89049fafe149d535d584e7d4cd2497c298c16043 Reviewed-on: http://gerrit.cloudera.org:8080/14862 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/fs/CMakeLists.txt M src/kudu/fs/block_manager-test.cc M src/kudu/fs/data_dirs.cc M src/kudu/fs/data_dirs.h R src/kudu/fs/dir_util-test.cc R src/kudu/fs/dir_util.cc R src/kudu/fs/dir_util.h M src/kudu/fs/error_manager.h M src/kudu/fs/file_block_manager.cc M src/kudu/fs/fs.proto M src/kudu/fs/fs_manager-test.cc M src/kudu/fs/log_block_manager.cc M src/kudu/fs/log_block_manager.h 13 files changed, 123 insertions(+), 129 deletions(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/14862 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I89049fafe149d535d584e7d4cd2497c298c16043 Gerrit-Change-Number: 14862 Gerrit-PatchSet: 4 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241)
