You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Lars Volker (Code Review)" <ge...@cloudera.org> on 2016/05/03 15:01:48 UTC

[Impala-CR](cdh5-trunk) Rename FilesystemUtil::CreateDirectory()

Lars Volker has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/2935

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
---
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(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/35/2935/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2935
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8baf28c09a4e400276a4bac8c6c3c97ccbce18a9
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Lars Volker <lv...@cloudera.com>