[GitHub] storm pull request #2634: [STORM-3021] Fix wrong usages of Config.TOPOLOGY_W...

2018-04-12 Thread Ethanlm
GitHub user Ethanlm opened a pull request: https://github.com/apache/storm/pull/2634 [STORM-3021] Fix wrong usages of Config.TOPOLOGY_WORKERS on RAS cluster https://issues.apache.org/jira/browse/STORM-3021 The Resource Aware Scheduler doesn't honor `Config.TOPOLOGY_WORKERS`.

[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 ---

Send (Netty Client) metrics

2018-04-12 Thread Walid Aljoby
Hi, I am collecting some metrics via registering through "Metrics Consumer".Most user-defined and system metrics like queues, _receiver-iconnection are available. However, I am also interested in Send (Netty Client) metric, called  __send-iconnection. But I could not fined it reported among other

[GitHub] storm pull request #2633: STORM-3028 HdfsSpout: handle empty file in case of...

2018-04-12 Thread ghajos
GitHub user ghajos opened a pull request: https://github.com/apache/storm/pull/2633 STORM-3028 HdfsSpout: handle empty file in case of ackers There is a problem with all the storm-hdfs junit tests. The test results may not be valid. You can merge this pull request into a Git reposi