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

Reply via email to