stevenzwu commented on code in PR #13832:
URL: https://github.com/apache/iceberg/pull/13832#discussion_r2281301974


##########
flink/v2.0/flink/src/main/java/org/apache/iceberg/flink/maintenance/api/FlinkMaintenanceConfig.java:
##########
@@ -31,27 +31,50 @@ public class FlinkMaintenanceConfig {
 
   public static final String PREFIX = "flink-maintenance.";
 
+  // Configuration for lock check delay (in seconds)
+  // This setting controls the delay between each lock check during the 
rewrite operation
   public static final String LOCK_CHECK_DELAY = PREFIX + 
"lock-check-delay-seconds";
   public static final ConfigOption<Long> LOCK_CHECK_DELAY_OPTION =
       ConfigOptions.key(LOCK_CHECK_DELAY)
           .longType()
-          .defaultValue(TableMaintenance.LOCK_CHECK_DELAY_SECOND_DEFAULT);
+          .defaultValue(TableMaintenance.LOCK_CHECK_DELAY_SECOND_DEFAULT)
+          .withDescription(
+              "The delay time (in seconds) between each lock check during the 
rewrite operation.");

Review Comment:
   is it only applicable to `rewrite operation"? what about other maintenance 
action like "expire snapshot"?



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to