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

ASF GitHub Bot commented on FLINK-5167:
---------------------------------------

Github user shijinkui commented on the issue:

    https://github.com/apache/flink/pull/2874
  
    > The PR changes several @Public and @PublicEvolving methods.
    hi, @fhueske 
    Today when set some config, find some function cant't chained one by one. 
So want to complete all the function needed `return this` conveniently. If 
chained setting config, the setting will be together, avoid setting config 
evrywhere.
    
    It's only better used, not important. You can close this. Failed building 
is noisy.


> StreamExecutionEnvironment's set function return `this` instead of void
> -----------------------------------------------------------------------
>
>                 Key: FLINK-5167
>                 URL: https://issues.apache.org/jira/browse/FLINK-5167
>             Project: Flink
>          Issue Type: Bug
>          Components: DataStream API
>            Reporter: shijinkui
>
> for example :
> public void setNumberOfExecutionRetries(int numberOfExecutionRetries) {
>       config.setNumberOfExecutionRetries(numberOfExecutionRetries);
> }
> change to:
> public StreamExecutionEnvironment setNumberOfExecutionRetries(int 
> numberOfExecutionRetries) {
>       config.setNumberOfExecutionRetries(numberOfExecutionRetries);
>       return this;
> }



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to