Hello Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3019
to look at the new patch set (#2).
Change subject: IMPALA-3517: S3: Get rid of hdfsDelete() warning message
......................................................................
IMPALA-3517: S3: Get rid of hdfsDelete() warning message
In the patch for IMPALA-3452, a bug was introduced which does an
unnecessary call to hdfsDelete() on the staging directory of a
partition(s), when the user selects the option of skipping the
staging step. Also, this delete will not happen when it should happen,
e.g on HDFS partitions. (Although it will be deleted by the
coordinator later).
This patch fixes this bug by making sure that we delete the staging
directory when it is necessary and skip the call to delete when we
do not want it.
Change-Id: I0a81ba0abfc24ee56689211579f46ac353e98adb
---
M be/src/exec/hdfs-table-sink.cc
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/19/3019/2
--
To view, visit http://gerrit.cloudera.org:8080/3019
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I0a81ba0abfc24ee56689211579f46ac353e98adb
Gerrit-PatchSet: 2
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Sailesh Mukil <[email protected]>
Gerrit-Reviewer: Dan Hecht <[email protected]>
