XComp commented on a change in pull request #19102:
URL: https://github.com/apache/flink/pull/19102#discussion_r827919525
##########
File path:
flink-core/src/main/java/org/apache/flink/configuration/CleanupOptions.java
##########
@@ -128,19 +134,23 @@ public static String extractAlphaNumericCharacters(String
paramName) {
public static final ConfigOption<Integer>
CLEANUP_STRATEGY_FIXED_DELAY_ATTEMPTS =
ConfigOptions.key(createFixedDelayParameterPrefix("attempts"))
.intType()
- .defaultValue(1)
+ .noDefaultValue()
Review comment:
because it would then print this a weirdly big number and I thought that
it's better to have an explicit no value instead.
--
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]
