You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@storm.apache.org by srdo <gi...@git.apache.org> on 2017/11/30 19:01:53 UTC

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

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 mean that all logging done to the log4j12 API is lost unless the user adds a log4j12 configuration file to Storm.


---