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

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

Github user HeartSaVioR commented on the pull request:

    https://github.com/apache/storm/pull/309#issuecomment-62289228
  
    @SeanTAllen @harshach 
    Hey, I've added unit test to PR branch.
    I've confirmed that fails on current master branch of Apache Storm and 
succeeds on current modification.
    Could you check it and merge to reflect current PR? Thanks!


> "topology.enable.message.timeouts" does nothing
> -----------------------------------------------
>
>                 Key: STORM-549
>                 URL: https://issues.apache.org/jira/browse/STORM-549
>             Project: Apache Storm
>          Issue Type: Bug
>    Affects Versions: 0.9.2-incubating, 0.9.1-incubating
>            Reporter: Sean T Allen
>             Fix For: 0.9.3-rc2
>
>
> The configuration option, added on 0.8.0 was never implemented:
> ➜  apache-storm-0.9.2-incubating ag "topology.enable.message.timeouts"
> conf/defaults.yaml
> 122:topology.enable.message.timeouts: true
> storm-core/src/jvm/backtype/storm/Config.java
> 512:    public static final String TOPOLOGY_ENABLE_MESSAGE_TIMEOUTS = 
> "topology.enable.message.timeouts";
> ➜  apache-storm-0.9.2-incubating ag TOPOLOGY_ENABLE_MESSAGE_TIMEOUTS
> storm-core/src/jvm/backtype/storm/Config.java
> 512:    public static final String TOPOLOGY_ENABLE_MESSAGE_TIMEOUTS = 
> "topology.enable.message.timeouts";
> 513:    public static final Object TOPOLOGY_ENABLE_MESSAGE_TIMEOUTS_SCHEMA = 
> Boolean.class;



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

Reply via email to