Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/19570
Change subject: [fs] update on FsManager's constructor ...................................................................... [fs] update on FsManager's constructor While reviewing on of the recent patches, I noticed that the signature of FsManager's constructors might be improved to avoid specifying default instance of FsManagerOpts as the second parameter. This patch addresses that and also cleans up instantiation of FsManagerOpts in a couple of places in kudu CLI tools. This patch doesn't contain any functional modifications. Change-Id: Ib76723a230142a4903cca4f47fbde44afa15abed --- M src/kudu/fs/fs_manager.h M src/kudu/master/master_runner.cc M src/kudu/tools/tool_action_fs.cc M src/kudu/tools/tool_action_local_replica.cc M src/kudu/tools/tool_action_perf.cc 5 files changed, 8 insertions(+), 8 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/70/19570/1 -- To view, visit http://gerrit.cloudera.org:8080/19570 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ib76723a230142a4903cca4f47fbde44afa15abed Gerrit-Change-Number: 19570 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
