Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15737 )
Change subject: IMPALA-9653: Impala shouldn't create/remove staging directory during transactional INSERTs ...................................................................... IMPALA-9653: Impala shouldn't create/remove staging directory during transactional INSERTs This commit eliminates an unnecessary hdfsDelete() call for INSERT-only tables. For such tables we don't create a staging directory, hence there is nothing to delete. It also eliminates a confusing log message about the deletion of the staging directory (which did not even exist). I also added a DCHECK to check that we don't even set the staging directory for such tables. Change-Id: Ie6a3a502fb3b3cfe7a68323d3b0145e5fb149460 Reviewed-on: http://gerrit.cloudera.org:8080/15737 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/runtime/coordinator.cc M be/src/runtime/coordinator.h M fe/src/main/java/org/apache/impala/service/Frontend.java 3 files changed, 19 insertions(+), 4 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/15737 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ie6a3a502fb3b3cfe7a68323d3b0145e5fb149460 Gerrit-Change-Number: 15737 Gerrit-PatchSet: 5 Gerrit-Owner: Zoltan Borok-Nagy <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
