[jira] [Commented] (STORM-578) Calls to submit-mocked-assignment in supervisor-test use invalid executor-id format

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

ASF GitHub Bot commented on STORM-578:
--------------------------------------

Github user lightyang commented on the pull request:

    https://github.com/apache/storm/pull/319#issuecomment-64975717
  
    Thanks, @revans2
    I've corrected the other calls.
    This did lead to test failures in my test runs, though the place you mentioned didn't seem to cause any issue (I suspect executor-id->tasks isn't call in that particular test).
    JIRA: https://issues.apache.org/jira/browse/STORM-578


> Calls to submit-mocked-assignment in supervisor-test use invalid executor-id format
> -----------------------------------------------------------------------------------
>
>                 Key: STORM-578
>                 URL: https://issues.apache.org/jira/browse/STORM-578
>             Project: Apache Storm
>          Issue Type: Bug
>         Environment: Ubuntu 14.04
>            Reporter: Mansheng Yang
>            Assignee: Mansheng Yang
>            Priority: Minor
>              Labels: test
>
> Calls to submit-mocked-assignment in supervisor-test are using invalid executor-id format, e.g.:
> {noformat}
> {
> [1] ["sup1" 1]
> [2] ["sup1" 2] 
> [3] ["sup2" 1] 
> [4] ["sup2" 1]
> }
> {noformat}
> Since executor ids should be of the form \[first-task last-task], the correct argument should be
> {noformat}
> {
> [1 1] ["sup1" 1] 
> [2 2] ["sup1" 2] 
> [3 3] ["sup2" 1] 
> [4 4] ["sup2" 1]
> }
> {noformat}
> See https://github.com/apache/storm/pull/319



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