Myasuka commented on a change in pull request #16153:
URL: https://github.com/apache/flink/pull/16153#discussion_r654247587
##########
File path:
flink-core/src/main/java/org/apache/flink/configuration/CheckpointingOptions.java
##########
@@ -102,13 +102,15 @@
/** Whether to enable state change log. */
@Documentation.Section(value =
Documentation.Sections.COMMON_STATE_BACKENDS)
- @Documentation.ExcludeFromDocumentation("Hidden for now")
public static final ConfigOption<Boolean> ENABLE_STATE_CHANGE_LOG =
ConfigOptions.key("state.backend.changelog.enabled")
.booleanType()
- .defaultValue(false)
+ .noDefaultValue()
Review comment:
As discussed offline, this configuration should have a default value of
`false`. We might have different kinds of implementation but we should tell
user explicitly that this option is disabled by default.
--
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
