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

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

GitHub user zentol opened a pull request:

    https://github.com/apache/flink/pull/4800

    [FLINK-7808] [REST] JobDetails constructor checks size of tasksPerSta…

    …te argument
    ## What is the purpose of the change
    
    This PR modifies the JobDetails constructor to verify that the 
`tasksPerState` array size is equal to `ExecutionState.values().length`. The 
array contains the number of tasks in each execution state, and it thus 
required that the sizes match.


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/zentol/flink 7808

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/flink/pull/4800.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #4800
    
----
commit b61f951cd6d72631f50607a6fa10a314a13bb3fb
Author: zentol <ches...@apache.org>
Date:   2017-10-11T11:47:38Z

    [FLINK-7808] [REST] JobDetails constructor checks size of tasksPerState 
argument

----


> JobDetails constructor should check length of tasksPerState array
> -----------------------------------------------------------------
>
>                 Key: FLINK-7808
>                 URL: https://issues.apache.org/jira/browse/FLINK-7808
>             Project: Flink
>          Issue Type: Bug
>          Components: REST
>    Affects Versions: 1.4.0
>            Reporter: Chesnay Schepler
>            Assignee: Chesnay Schepler
>            Priority: Minor
>             Fix For: 1.4.0
>
>
> The JobDetails constructor accepts an {{int[] tasksPerState}} argument, which 
> represents the number of tasks in each {{ExecutionState}}. There is no check 
> in place to verify that the array has the correct size, which the json 
> serializer assumes to be the case.



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

Reply via email to