akune commented on code in PR #14603:
URL: https://github.com/apache/iceberg/pull/14603#discussion_r2534486516
##########
core/src/main/java/org/apache/iceberg/jdbc/JdbcUtil.java:
##########
@@ -42,6 +42,9 @@ final class JdbcUtil {
// property to control if catalog tables are created during initialization
static final String INIT_CATALOG_TABLES_PROPERTY =
JdbcCatalog.PROPERTY_PREFIX + "init-catalog-tables";
+ // property to control if catalog schema is updated if necessary during
initialization
+ static final String UPDATE_CATALOG_SCHEMA_IF_NECESSARY_PROPERTY =
+ JdbcCatalog.PROPERTY_PREFIX + "update-catalog-schema-if-necessary";
Review Comment:
I completely understand and agree, that would be the best approach.
Would you still consider an option to skip checking and updating the schema
(especially when the user already opted out of having the tables created)
through JdbcCatalog? I would consider that useful because in production
scenarios the database user might not have DDL permissions.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
