storm-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (STORM-354) Allow users to pass TEST-TIMEOUT-MS as param for complete-topology
Date Tue, 08 Jul 2014 23:41:05 GMT

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

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

Github user nsj007 commented on the pull request:

    https://github.com/apache/incubator-storm/pull/122#issuecomment-48413339
  
    Is there any progress on this. The 5000ms timeout pretty much fails all my test when upgrading
from STORM 0.9.01 to apache-storm-0.9.2-incubator 


> Allow users to pass TEST-TIMEOUT-MS as param for complete-topology
> ------------------------------------------------------------------
>
>                 Key: STORM-354
>                 URL: https://issues.apache.org/jira/browse/STORM-354
>             Project: Apache Storm (Incubating)
>          Issue Type: Improvement
>    Affects Versions: 0.9.2-incubating
>            Reporter: Aaron Levin
>            Priority: Minor
>              Labels: test
>             Fix For: 0.9.2-incubating
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> It would be nice if complete-topology allowed a user to pass in the default timeout as
a parameter (rather than just setting it as 5000ms). I had a test that kept failing by taking
too long.
> This PR adds functionality without breaking any existing code. Tests pass.
> (PS - not sure if this is the proper way to submit a PR for an apache project. I'm open
to feedback. Thanks)



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

Mime
View raw message