Wenzhe Zhou has uploaded a new patch set (#9). ( http://gerrit.cloudera.org:8080/20971 )
Change subject: IMPALA-12378: Auto Ship JDBC Data Source ...................................................................... IMPALA-12378: Auto Ship JDBC Data Source This patch moves the source files of jdbc package to fe. Data source location is optional. Data source could be created without specifying HDFS location. Assume data source class is in the classpath and instance of data source class could be created with current class loader. Impala still try to load the jar file of the data source in runtime if it's set in data source location. Testing: - Passed core test - Passed dockerised-tests Change-Id: I0daff8db6231f161ec27b45b51d78e21733d9b1f --- M be/src/exec/external-data-source-executor.cc M bin/impala-config.sh M bin/rat_exclude_files.txt M fe/pom.xml M fe/src/compat-hive-3/java/org/apache/impala/compat/MetastoreShim.java M fe/src/main/cup/sql-parser.cup M fe/src/main/java/org/apache/impala/analysis/CreateDataSrcStmt.java M fe/src/main/java/org/apache/impala/analysis/CreateTableDataSrcStmt.java M fe/src/main/java/org/apache/impala/extdatasource/ExternalDataSourceExecutor.java R fe/src/main/java/org/apache/impala/extdatasource/jdbc/JdbcDataSource.java R fe/src/main/java/org/apache/impala/extdatasource/jdbc/README.md R fe/src/main/java/org/apache/impala/extdatasource/jdbc/conf/DatabaseType.java R fe/src/main/java/org/apache/impala/extdatasource/jdbc/conf/JdbcStorageConfig.java R fe/src/main/java/org/apache/impala/extdatasource/jdbc/conf/JdbcStorageConfigManager.java R fe/src/main/java/org/apache/impala/extdatasource/jdbc/dao/DB2DatabaseAccessor.java R fe/src/main/java/org/apache/impala/extdatasource/jdbc/dao/DatabaseAccessor.java R fe/src/main/java/org/apache/impala/extdatasource/jdbc/dao/DatabaseAccessorFactory.java R fe/src/main/java/org/apache/impala/extdatasource/jdbc/dao/GenericJdbcDatabaseAccessor.java R fe/src/main/java/org/apache/impala/extdatasource/jdbc/dao/ImpalaDatabaseAccessor.java R fe/src/main/java/org/apache/impala/extdatasource/jdbc/dao/JdbcRecordIterator.java R fe/src/main/java/org/apache/impala/extdatasource/jdbc/dao/JethroDatabaseAccessor.java R fe/src/main/java/org/apache/impala/extdatasource/jdbc/dao/MsSqlDatabaseAccessor.java R fe/src/main/java/org/apache/impala/extdatasource/jdbc/dao/MySqlDatabaseAccessor.java R fe/src/main/java/org/apache/impala/extdatasource/jdbc/dao/OracleDatabaseAccessor.java R fe/src/main/java/org/apache/impala/extdatasource/jdbc/dao/PostgresDatabaseAccessor.java R fe/src/main/java/org/apache/impala/extdatasource/jdbc/exception/JdbcDatabaseAccessException.java R fe/src/main/java/org/apache/impala/extdatasource/jdbc/util/QueryConditionUtil.java M fe/src/main/java/org/apache/impala/planner/DataSourceScanNode.java M fe/src/test/java/org/apache/impala/analysis/AnalyzeDDLTest.java M fe/src/test/java/org/apache/impala/analysis/ParserTest.java M fe/src/test/java/org/apache/impala/customcluster/LdapHS2Test.java M java/ext-data-source/pom.xml M java/pom.xml M testdata/bin/copy-ext-data-sources.sh M testdata/workloads/functional-query/queries/QueryTest/impala-ext-jdbc-tables-predicates.test M testdata/workloads/functional-query/queries/QueryTest/impala-ext-jdbc-tables.test M testdata/workloads/functional-query/queries/QueryTest/jdbc-data-source-with-keystore.test M testdata/workloads/functional-query/queries/QueryTest/jdbc-data-source.test M testdata/workloads/functional-query/queries/QueryTest/mysql-ext-jdbc-tables.test 39 files changed, 79 insertions(+), 59 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/71/20971/9 -- To view, visit http://gerrit.cloudera.org:8080/20971 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I0daff8db6231f161ec27b45b51d78e21733d9b1f Gerrit-Change-Number: 20971 Gerrit-PatchSet: 9 Gerrit-Owner: Wenzhe Zhou <wz...@cloudera.com> Gerrit-Reviewer: Abhishek Rawat <ara...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Wenzhe Zhou <wz...@cloudera.com> Gerrit-Reviewer: gaurav singh <gsi...@cloudera.com>
