Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8399 )
Change subject: fs: ensure the existence of important directories on open ...................................................................... fs: ensure the existence of important directories on open Seems like a pretty easy way to improve robustness, especially for CLI tools that may not access these directories directly but nevertheless want to guarantee their existence before making destructive changes. Change-Id: I5238987273886421f8e6f87f850183dc81c8c4ec Reviewed-on: http://gerrit.cloudera.org:8080/8399 Tested-by: Adar Dembo <a...@cloudera.com> Reviewed-by: Andrew Wong <aw...@cloudera.com> Reviewed-by: Todd Lipcon <t...@apache.org> --- M src/kudu/fs/fs_manager-test.cc M src/kudu/fs/fs_manager.cc M src/kudu/tools/kudu-tool-test.cc 3 files changed, 38 insertions(+), 6 deletions(-) Approvals: Adar Dembo: Verified Andrew Wong: Looks good to me, but someone else must approve Todd Lipcon: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/8399 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I5238987273886421f8e6f87f850183dc81c8c4ec Gerrit-Change-Number: 8399 Gerrit-PatchSet: 4 Gerrit-Owner: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org>