impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sailesh Mukil (Code Review)" <ger...@cloudera.org>
Subject [Impala-CR](cdh5-trunk) IMPALA-3517: S3: Get rid of hdfsDelete() warning message
Date Tue, 10 May 2016 23:03:58 GMT
Sailesh Mukil has uploaded a new change for review.

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

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/1
-- 
To view, visit http://gerrit.cloudera.org:8080/3019
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0a81ba0abfc24ee56689211579f46ac353e98adb
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Sailesh Mukil <sailesh@cloudera.com>

Mime
View raw message