Hello Tidy Bot, Kudu Jenkins, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14862
to look at the new patch set (#3).
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
---
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(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/62/14862/3
--
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: newpatchset
Gerrit-Change-Id: I89049fafe149d535d584e7d4cd2497c298c16043
Gerrit-Change-Number: 14862
Gerrit-PatchSet: 3
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)
