singhpk234 commented on code in PR #1371:
URL: https://github.com/apache/polaris/pull/1371#discussion_r2048006676
##########
extension/persistence/relational-jdbc/src/main/java/org/apache/polaris/extension/persistence/relational/jdbc/DatasourceOperations.java:
##########
@@ -171,6 +181,20 @@ public boolean isAlreadyExistsException(SQLException e) {
}
private Connection borrowConnection() throws SQLException {
- return datasource.getConnection();
+ Connection connection = datasource.getConnection();
+ if (connection
+ .getMetaData()
+ .getDatabaseProductName()
+ .toLowerCase(Locale.ROOT)
+ .equals("postgresql")) {
Review Comment:
sounds good, i think i will need to think this more, this work-around mainly
came from the fact that connection.setSchema() doesn't work for postgres
Drviers, may be I can refactor these to a connection manager class which
abstracts that away ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
