rdblue commented on code in PR #6432:
URL: https://github.com/apache/iceberg/pull/6432#discussion_r1055831616
##########
core/src/main/java/org/apache/iceberg/SystemProperties.java:
##########
@@ -41,7 +41,10 @@ private SystemProperties() {}
public static final int IO_MANIFEST_CACHE_MAX_FILEIO_DEFAULT = 8;
- static boolean getBoolean(String systemProperty, boolean defaultValue) {
+ public static final String DELETE_POS_FILES_THREADS_ENABLED =
Review Comment:
I think this should be an engine option rather than a System property.
@aokolnychyi what do you think?
--
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
