Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/20833
to look at the new patch set (#2).
Change subject: [fs] Simplify the data directory inheritance hierarchy
......................................................................
[fs] Simplify the data directory inheritance hierarchy
This patch doesn't introduce functional changes, but
only simplify the inheritance hierarchy of class Dir
to reduce the complex when extend the Dir class, for
example, when adding a sub-class of Dir to contain a
RocksDB instance, see http://gerrit.cloudera.org:8080/18569.
Because there are not sufficient reasons, so the
DataDir sub-class is removed and the implementation
functions are moved to the super class Dir.
Change-Id: I2d7059e32c7fd8444256ea031bebd153b318a5a3
---
M src/kudu/fs/data_dirs.cc
M src/kudu/fs/data_dirs.h
M src/kudu/fs/dir_manager.cc
M src/kudu/fs/dir_manager.h
M src/kudu/fs/fs_manager.h
5 files changed, 19 insertions(+), 38 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/33/20833/2
--
To view, visit http://gerrit.cloudera.org:8080/20833
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I2d7059e32c7fd8444256ea031bebd153b318a5a3
Gerrit-Change-Number: 20833
Gerrit-PatchSet: 2
Gerrit-Owner: Yingchun Lai <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
