Github user Ethanlm commented on the issue:
https://github.com/apache/storm/pull/2441
Thanks @revans2 for the reply
---
Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/2441
@srdo I agree that we don't want slf4j-log4j12 on the classpath whenever
possible. The use case we have is adding hadoop to our daemon classpath with
all of the configs and everything else needed to
Github user govind-menon commented on the issue:
https://github.com/apache/storm/pull/2437
@revans2 There is now - https://issues.apache.org/jira/browse/STORM-2840
---
GitHub user revans2 opened a pull request:
https://github.com/apache/storm/pull/2442
STORM-2837: ConstraintSolverStrategy
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/revans2/incubator-storm STORM-2837
Alternatively you can r
Github user srdo commented on the issue:
https://github.com/apache/storm/pull/2440
+1, nice catch
---
Github user srdo commented on the issue:
https://github.com/apache/storm/pull/2441
I think this is the wrong approach. We should exclude slf4j-log4j12 from
whichever dependency is including it instead. We want all logging to go to
log4j 2 and our worker log, making this change would m
GitHub user Ethanlm opened a pull request:
https://github.com/apache/storm/pull/2441
[STORM-2838] Replace log4j-over-slf4j with log4j-1.2-api
https://issues.apache.org/jira/browse/STORM-2838
To let storm work with HDFS, we need to replace log4j-over-slf4j with
log4j-1.2-api
