Internal Jenkins has submitted this change and it was merged. 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 Reviewed-on: http://gerrit.cloudera.org:8080/3019 Reviewed-by: Sailesh Mukil <[email protected]> Tested-by: Internal Jenkins --- M be/src/exec/hdfs-table-sink.cc 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Internal Jenkins: Verified Sailesh Mukil: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/3019 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0a81ba0abfc24ee56689211579f46ac353e98adb Gerrit-PatchSet: 3 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Sailesh Mukil <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Sailesh Mukil <[email protected]>
