JunRuiLee commented on code in PR #23758:
URL: https://github.com/apache/flink/pull/23758#discussion_r1400088360
##########
flink-core/src/main/java/org/apache/flink/api/common/restartstrategy/RestartStrategies.java:
##########
@@ -111,7 +118,17 @@ public static ExponentialDelayRestartStrategyConfiguration
exponentialDelayResta
initialBackoff, maxBackoff, backoffMultiplier,
resetBackoffThreshold, jitterFactor);
}
- /** Abstract configuration for restart strategies. */
+ /**
+ * Abstract configuration for restart strategies.
+ *
+ * @deprecated The {@link RestartStrategyConfiguration} class is marked as
deprecated because
+ * starting from Flink 1.19, all complex Java objects related to
configuration should be
+ * replaced by ConfigOption. In a future major version of Flink, this
class will be removed
+ * entirely. It is recommended to switch to using the ConfigOptions
provided by {@link
+ * org.apache.flink.configuration.RestartStrategyOptions} for
configuring restart
+ * strategies.
+ */
+ @Deprecated
Review Comment:
You're right, it's not necessary since the whole class is deprecated. I've
made the changes 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]
