cobed95 commented on code in PR #4945: URL: https://github.com/apache/polaris/pull/4945#discussion_r3662160595
########## runtime/admin/src/main/resources/application.properties: ########## @@ -47,6 +47,13 @@ quarkus.mongodb.devservices.enabled=false # - relational-jdbc - JdbcMetaStoreManagerFactory # - nosql - NoSQL persistence backend, define the backend type via 'polaris.persistence.nosql.backend' polaris.persistence.type=relational-jdbc +# The database schema holding the Polaris tables for the 'relational-jdbc' persistence type. The +# schema is selected by the JDBC driver on each connection (the PostgreSQL driver's currentSchema +# connection property, also used for CockroachDB) and must match the schema configured on the +# running Polaris server. Override this property, or set currentSchema directly in the JDBC URL +# (the URL takes precedence). The schema must exist before bootstrapping; creating it is a DBA +# task requiring elevated privileges. +quarkus.datasource.jdbc.additional-jdbc-properties.currentSchema=POLARIS_SCHEMA Review Comment: Hi @singhpk234 , this property works only for PG and CockroachDB and is a no-op for MySQL + etc. as mentioned above. My rationale was that MySQL would define the schema in the JDBC URL anyway (as PG / CockroachDB should for database), so no config change is needed if we start supporting MySQL. -- 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]
