1996fanrui commented on code in PR #22984: URL: https://github.com/apache/flink/pull/22984#discussion_r1442471228
########## flink-core/src/main/java/org/apache/flink/api/common/restartstrategy/RestartStrategies.java: ########## @@ -72,9 +72,23 @@ public static RestartStrategyConfiguration fixedDelayRestart( * @param restartAttempts Number of restart attempts for the FixedDelayRestartStrategy * @param delayInterval Delay in-between restart attempts for the FixedDelayRestartStrategy * @return FixedDelayRestartStrategy + * @deprecated Use {@link #fixedDelayRestart(int, Duration)} */ + @Deprecated Review Comment: You are right, this change is useful if FLIP-381 is reverted. But it's very unlikely. And these changes can be done when it happens. So far, this change isn't required, and I'm fine to have it. So up to you. -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org