Joe McDonnell has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/23658 )

Change subject: IMPALA-14545: Don't use absolute hdfs paths for JDBC table 
driver.url
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/23658/1/testdata/bin/create-ext-data-source-table.sql
File testdata/bin/create-ext-data-source-table.sql:

http://gerrit.cloudera.org:8080/#/c/23658/1/testdata/bin/create-ext-data-source-table.sql@90
PS1, Line 90: 
"driver.url"="/test-warehouse/data-sources/jdbc-drivers/postgresql-jdbc.jar",
> How does this interact with Ozone testing?
It's a bit weird. There are end to end tests that use tables of the same name, 
but they all use a unique database and create/drop the table as part of the 
test. Those tests use FILESYSTEM_PREFIX and should be fine with Ozone. I'm 
about to file a JIRA about one that isn't, but that doesn't intersect with this 
file.

These tables are only used for frontend tests, and Ozone doesn't run frontend 
tests. If we wanted to do that, these would need to use FILESYTEM_PREFIX. 
That's another way to fix this.

Also, some of these tables don't seem to be used at all, so I should look into 
removing those and see if it works.



--
To view, visit http://gerrit.cloudera.org:8080/23658
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3f2c86fcc6c1dee75d7d9a9be04468cb197ae13c
Gerrit-Change-Number: 23658
Gerrit-PatchSet: 1
Gerrit-Owner: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Comment-Date: Mon, 10 Nov 2025 22:45:12 +0000
Gerrit-HasComments: Yes

Reply via email to