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

Reply via email to