[GitHub] storm issue #2000: STORM-2411 Setting topology.eventlogger.executors=0 in de...

2018-04-13 Thread roshannaik
Github user roshannaik commented on the issue: https://github.com/apache/storm/pull/2000 thanks. ---

[GitHub] storm issue #2000: STORM-2411 Setting topology.eventlogger.executors=0 in de...

2018-04-12 Thread Ethanlm
Github user Ethanlm commented on the issue: https://github.com/apache/storm/pull/2000 I think I can include it in my PR for https://issues.apache.org/jira/browse/STORM-3021 ---

[GitHub] storm issue #2000: STORM-2411 Setting topology.eventlogger.executors=0 in de...

2018-04-12 Thread roshannaik
Github user roshannaik commented on the issue: https://github.com/apache/storm/pull/2000 Good point. Will provide a fix. ---

[GitHub] storm issue #2000: STORM-2411 Setting topology.eventlogger.executors=0 in de...

2018-04-12 Thread Ethanlm
Github user Ethanlm commented on the issue: https://github.com/apache/storm/pull/2000 Just realized that https://github.com/apache/storm/blob/master/storm-client/src/jvm/org/apache/storm/Config.java#L398 should be updated because of this patch ---

[GitHub] storm issue #2000: STORM-2411 Setting topology.eventlogger.executors=0 in de...

2017-03-20 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/2000 +1 This makes consistent with 1.x branch. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this