Myasuka commented on a change in pull request #15119:
URL: https://github.com/apache/flink/pull/15119#discussion_r604818088
##########
File path:
flink-core/src/main/java/org/apache/flink/configuration/CheckpointingOptions.java
##########
@@ -39,8 +39,12 @@
* StateBackendFactory#createFromConfig(ReadableConfig, ClassLoader)}
method is called.
*
* <p>Recognized shortcut names are 'hashmap' and 'rocksdb'.
+ *
+ * @deprecated Use {@link StateBackendOptions#STATE_BACKEND}.
*/
- @Documentation.Section(value =
Documentation.Sections.COMMON_STATE_BACKENDS, position = 1)
+ @Documentation.Section(value =
Documentation.Sections.COMMON_STATE_BACKENDS)
+ @Documentation.ExcludeFromDocumentation("Hidden for deprecated")
+ @Deprecated
public static final ConfigOption<String> STATE_BACKEND =
Review comment:
I think it's okay to refactor like this to deduplicate.
BTW. @carp84 also suggested this change offline yesterday but agreed that
such change is not something must do.
--
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:
[email protected]
