Riza Suminto has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23787 )
Change subject: IMPALA-14546: Fix jdbc driver urls in test_postgres_jdbc_tables ...................................................................... IMPALA-14546: Fix jdbc driver urls in test_postgres_jdbc_tables TestPostgresJdbcTables.test_postgres_jdbc_tables uses hardcoded paths for JDBC driver URLs: "/test-warehouse/data-sources/jdbc-drivers/postgresql-jdbc.jar". This doesn't work correctly when running on Ozone where we need the prefix of "ofs://localhost:9862/impala". This patch fixes the issue by constructing the driver URL with FILESYSTEM_PREFIX which is "ofs://localhost:9862/impala" on Ozone. See more in bin/impala-config.sh about how it's set for different filesystems. Tests: - Ran the test on Ozone. Change-Id: Ie0c4368b3262d4dcb9e1c05475506411be2e2ef5 Reviewed-on: http://gerrit.cloudera.org:8080/23787 Tested-by: Impala Public Jenkins <[email protected]> Reviewed-by: Riza Suminto <[email protected]> --- M tests/custom_cluster/test_ext_data_sources.py 1 file changed, 9 insertions(+), 6 deletions(-) Approvals: Impala Public Jenkins: Verified Riza Suminto: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/23787 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ie0c4368b3262d4dcb9e1c05475506411be2e2ef5 Gerrit-Change-Number: 23787 Gerrit-PatchSet: 2 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Pranav Lodha <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]>
