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

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

Github user ptgoetz commented on the pull request:

    https://github.com/apache/storm/pull/279#issuecomment-61857050
  
    -1
    
    Per my previous comment, this needs to be an environment variable, not a 
system property. The problem is that the way that the clojure unit tests are 
run, system properties specified with the `-D` flag will not get propagated 
across JVMs. Environment variables will.
    
    @clockfly can you revert the merge?


> Test timeout should be configurable
> -----------------------------------
>
>                 Key: STORM-492
>                 URL: https://issues.apache.org/jira/browse/STORM-492
>             Project: Apache Storm
>          Issue Type: Bug
>            Reporter: Devika Nair
>            Assignee: Jungtaek Lim
>              Labels: newbie
>             Fix For: 0.9.3-rc2
>
>
> Test timeout is hard coded to 5000ms in 
> https://github.com/apache/incubator-storm/blob/master/storm-core/src/clj/backtype/storm/testing.clj.
>  Provide a way to override this value in order to handle longer running tests.



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

Reply via email to