[ 
https://issues.apache.org/jira/browse/FLINK-33592?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17787624#comment-17787624
 ] 

Junrui Li commented on FLINK-33592:
-----------------------------------

Hi [~zhangtao9876] ,

The community has already accepted FLIP-381: Deprecate configuration 
getters/setters that return/set complex Java objects 
([here|https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=278464992]).
 Therefore, it is not recommended to modify this method, and it is also 
recommended to use ConfigOption instead of the setter API. Additionally, since 
the setRestartStrategy method is annotated by @PublicEnvolving, any 
modifications to it would require community discussion.

> The return type of the function is void,not convenient to use
> -------------------------------------------------------------
>
>                 Key: FLINK-33592
>                 URL: https://issues.apache.org/jira/browse/FLINK-33592
>             Project: Flink
>          Issue Type: Improvement
>          Components: API / DataStream
>            Reporter: ZhangTao
>            Priority: Minor
>              Labels: pull-request-available
>
> {code:java}
> @PublicEvolving
> public void setRestartStrategy(
>         RestartStrategies.RestartStrategyConfiguration 
> restartStrategyConfiguration) {
>     config.setRestartStrategy(restartStrategyConfiguration);
> } {code}
> StreamExecutionEnvironment usually has many parameters that need to be 
> set.The return type is void, making it inconvenient to use.
> Others set methods 'return this; ' , only this method has a void return type.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to