rbalamohan commented on code in PR #6432: URL: https://github.com/apache/iceberg/pull/6432#discussion_r1059916781
########## 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: Using sys properties similar to worker threads in ThreadPools codebase. Making engine option wasn't straight forward via Deletes codebase. It may be easier to start with sys properties (similar to worker threads) and move to other mode if needed later?. -- 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