Wenzhe Zhou has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20710 )
Change subject: IMPALA-12471: Add unit tests of external jdbc tables for MySQL ...................................................................... IMPALA-12471: Add unit tests of external jdbc tables for MySQL This patch adds MySql tests for the "external data source" mechanism in Impala to implement data source for querying JDBC. This patch also fixes the handling of case-sensitive table and column names for MySQL query. Testing: - Added unit test for mysql and ran unit-test with JDBC driver mysql-connector-j-8.1.0.jar. This test requires to add the docker to sudoer's group. Also, the test is only run in 'exhaustive' mode. Change-Id: I446ec3d4ebaf53c8edac0b2d181514bde587dfae Reviewed-on: http://gerrit.cloudera.org:8080/20710 Tested-by: Impala Public Jenkins <[email protected]> Reviewed-by: Wenzhe Zhou <[email protected]> --- M java/ext-data-source/jdbc/src/main/java/org/apache/impala/extdatasource/jdbc/JdbcDataSource.java M java/ext-data-source/jdbc/src/main/java/org/apache/impala/extdatasource/jdbc/dao/DatabaseAccessor.java M java/ext-data-source/jdbc/src/main/java/org/apache/impala/extdatasource/jdbc/dao/GenericJdbcDatabaseAccessor.java M java/ext-data-source/jdbc/src/main/java/org/apache/impala/extdatasource/jdbc/dao/PostgresDatabaseAccessor.java A testdata/bin/clean-mysql-env.sh M testdata/bin/load-ext-data-sources.sh A testdata/bin/setup-mysql-env.sh A testdata/workloads/functional-query/queries/QueryTest/mysql-ext-jdbc-tables.test M tests/custom_cluster/test_ext_data_sources.py 9 files changed, 463 insertions(+), 13 deletions(-) Approvals: Impala Public Jenkins: Verified Wenzhe Zhou: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/20710 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I446ec3d4ebaf53c8edac0b2d181514bde587dfae Gerrit-Change-Number: 20710 Gerrit-PatchSet: 26 Gerrit-Owner: Anonymous Coward <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Wenzhe Zhou <[email protected]>
