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

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

Github user xccui commented on the issue:

    https://github.com/apache/flink/pull/5192
  
    Hi @aljoscha, I wonder if you could help review this PR when you are 
convenient. 
    
    Thanks, Xingcan


> Unify the value checks for setParallelism()
> -------------------------------------------
>
>                 Key: FLINK-8257
>                 URL: https://issues.apache.org/jira/browse/FLINK-8257
>             Project: Flink
>          Issue Type: Improvement
>          Components: Configuration
>            Reporter: Xingcan Cui
>            Assignee: Xingcan Cui
>
> The {{setParallelism()}} method exist in many components from different 
> levels. Some of the methods require the input value to be greater than {{1}} 
> (e.g., {{StreamTransformation.setParallelism()}}), while some of them also 
> allow the value to be {{ExecutionConfig.PARALLELISM_DEFAULT}}, which is 
> {{-1}} by default (e.g., {{DataSink.setParallelism()}}). We need to unify the 
> value checks for these methods.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to