[GitHub] storm issue #2441: [STORM-2838] Replace log4j-over-slf4j with log4j-1.2-api

2018-03-08 Thread Ethanlm
Github user Ethanlm commented on the issue: https://github.com/apache/storm/pull/2441 @srdo Thanks for the review. We decided to set the configs to only point to the existing hadoop config files (no jars from existing hadoop) to avoid this issue for now. Closing this until any fu

[GitHub] storm issue #2441: [STORM-2838] Replace log4j-over-slf4j with log4j-1.2-api

2017-12-01 Thread srdo
Github user srdo commented on the issue: https://github.com/apache/storm/pull/2441 Thanks for explaining @revans2. Please correct me if I get any of this wrong, I'm not very familiar with Hadoop: The issue is that you want the daemons to use HdfsBlobStore, which req

[GitHub] storm issue #2441: [STORM-2838] Replace log4j-over-slf4j with log4j-1.2-api

2017-11-30 Thread Ethanlm
Github user Ethanlm commented on the issue: https://github.com/apache/storm/pull/2441 Thanks @revans2 for the reply ---

[GitHub] storm issue #2441: [STORM-2838] Replace log4j-over-slf4j with log4j-1.2-api

2017-11-30 Thread revans2
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] storm issue #2441: [STORM-2838] Replace log4j-over-slf4j with log4j-1.2-api

2017-11-30 Thread srdo
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