[email protected] has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/20654 )
Change subject: IMPALA-12470 (PART-2): delete temporary file in /tmp after class loaded ...................................................................... IMPALA-12470 (PART-2): delete temporary file in /tmp after class loaded This patch fixes the bug added in the previous patch for IMPALA-12470. It adds the prefix "file://" to the unix standard path string to create the corresponding valid hadoop.fs.Path object. For example: "/tmp" is converted to "file:///tmp". Testing: 1. Deleted all the jar files in the /tmp directory. 2. Ran the local jdbc ext data sources tests: - impala-py.test tests/query_test/test_ext_data_sources.py - impala-py.test tests/custom_cluster/test_ext_data_sources.py 3. Upon completion of the tests successfully, Verified that there were no .jar files in the /tmp directory. Change-Id: Iab7cc66383bc62f209987dd3fb42fc3fc6604726 --- M fe/src/main/java/org/apache/impala/planner/DataSourceScanNode.java M java/ext-data-source/jdbc/src/main/java/org/apache/impala/extdatasource/jdbc/dao/GenericJdbcDatabaseAccessor.java 2 files changed, 2 insertions(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/54/20654/3 -- To view, visit http://gerrit.cloudera.org:8080/20654 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Iab7cc66383bc62f209987dd3fb42fc3fc6604726 Gerrit-Change-Number: 20654 Gerrit-PatchSet: 3 Gerrit-Owner: Anonymous Coward <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Wenzhe Zhou <[email protected]>
