Internal Jenkins has submitted this change and it was merged. Change subject: Rename FilesystemUtil::CreateDirectory() ......................................................................
Rename FilesystemUtil::CreateDirectory() CreateDirectory() will remove the target directory if it already exists. The old name did not reflect this behavior, and while the comment clearly said so, it caused confusion among several people. This change alters the name to make the behavior more obvious. Change-Id: I8baf28c09a4e400276a4bac8c6c3c97ccbce18a9 Reviewed-on: http://gerrit.cloudera.org:8080/2935 Reviewed-by: Dan Hecht <[email protected]> Tested-by: Internal Jenkins --- M be/src/runtime/buffered-block-mgr-test.cc M be/src/runtime/tmp-file-mgr-test.cc M be/src/runtime/tmp-file-mgr.cc M be/src/util/filesystem-util-test.cc M be/src/util/filesystem-util.cc M be/src/util/filesystem-util.h 6 files changed, 16 insertions(+), 16 deletions(-) Approvals: Internal Jenkins: Verified Dan Hecht: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/2935 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8baf28c09a4e400276a4bac8c6c3c97ccbce18a9 Gerrit-PatchSet: 2 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Lars Volker <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Tim Armstrong <[email protected]>
