ebyhr commented on code in PR #12893:
URL: https://github.com/apache/iceberg/pull/12893#discussion_r2059438422
##########
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/SparkSQLProperties.java:
##########
@@ -79,6 +79,11 @@ private SparkSQLProperties() {}
public static final String EXECUTOR_CACHE_ENABLED =
"spark.sql.iceberg.executor-cache.enabled";
public static final boolean EXECUTOR_CACHE_ENABLED_DEFAULT = true;
+ // Controls whether to enable executor cache for delete files
+ public static final String EXECUTOR_CACHE_DELETES_ENABLED =
+ "spark.sql.iceberg.executor-cache.deletes.enabled";
Review Comment:
`executor-cache.deletes.enabled` sounds like a flag to delete the executor
cache. How about renaming to `executor-cache.delete-files.enabled`?
--
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
