slfan1989 commented on code in PR #13832:
URL: https://github.com/apache/iceberg/pull/13832#discussion_r2295056399
##########
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:
@stevenzwu Thank you for your suggestion! I readed the relevant design
documents and found that this configuration is used in the following four
operations: `RewriteDataFiles`, `RewriteManifestFiles`, `ExpireSnapshots`, and
`DeleteOrphanFiles`. I have updated the description accordingly.
--
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]
