rdblue commented on code in PR #11317: URL: https://github.com/apache/iceberg/pull/11317#discussion_r1881170392
########## core/src/main/java/org/apache/iceberg/CatalogProperties.java: ########## @@ -78,6 +78,14 @@ private CatalogProperties() {} public static final boolean IO_MANIFEST_CACHE_ENABLED_DEFAULT = false; + /** + * Controls whether engines using a REST Catalog should delegate the drop table purge requests to the Catalog. + * Defaults to false, allowing the engine to use its own implementation for purging. + */ + public static final String REST_CATALOG_PURGE = "rest.catalog-purge"; Review Comment: Isn't the option whether to delegate to the catalog, not specifically REST? Why is the property specific to REST catalog? -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org