Zoltan Borok-Nagy has uploaded this change for review. ( 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 --- M be/src/runtime/coordinator.cc M fe/src/main/java/org/apache/impala/service/Frontend.java 2 files changed, 8 insertions(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/37/15737/1 -- 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: newchange Gerrit-Change-Id: Ie6a3a502fb3b3cfe7a68323d3b0145e5fb149460 Gerrit-Change-Number: 15737 Gerrit-PatchSet: 1 Gerrit-Owner: Zoltan Borok-Nagy <[email protected]>
