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

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

Github user d2r commented on the pull request:

    https://github.com/apache/incubator-storm/pull/193#issuecomment-49354550
  
    @iwasakims That is clear, thank you.  It is good that you tracked it down.
    
    I am +1 on this change.
    
    I will also file a JIRA to take a look at simulated time and mocking in 
this and other situations, because I think there may be a deeper issue with 
these kinds of tests particularly in light of another recent change to mitigate 
test hangs:
    
    
https://github.com/apache/incubator-storm/blob/ff345c1fa9dcbe55e96037bec3b59d06c3f64cd4/storm-core/src/clj/backtype/storm/testing.clj#L237-L240


> heartbeats-to-nimbus in supervisor-test failed due to unintentional worker 
> launch
> ---------------------------------------------------------------------------------
>
>                 Key: STORM-403
>                 URL: https://issues.apache.org/jira/browse/STORM-403
>             Project: Apache Storm (Incubating)
>          Issue Type: Bug
>            Reporter: Masatake Iwasaki
>            Priority: Minor
>
> heartbeats-to-nimbus in supervisor-test failed due to unintentional worker 
> launch. Moving the call of advance-cluster-time to inside 
> capture-changed-workers fixed this.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to