curcur commented on a change in pull request #14943: URL: https://github.com/apache/flink/pull/14943#discussion_r592267698
########## File path: flink-core/src/main/java/org/apache/flink/configuration/CheckpointingOptions.java ########## @@ -96,6 +96,16 @@ "Recognized shortcut names are 'jobmanager' and 'filesystem'.") .build()); + /** 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.enable-statechangelog") Review comment: I agreed with what Roman said, the state changelog is related to how the state is maintained and handled, and as a whole changing the behavior of checkpoint as well. "state.backend.changelog.enabled" yes, this is better. ---------------------------------------------------------------- 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