[GitHub] [spark] Kimahriman commented on pull request #37694: [SPARK-40246][BUILD] Make log4j-slf4j-impl always be a compile dependency

[GitBox <gi...@apache.org>]

Kimahriman commented on PR #37694:
URL: https://github.com/apache/spark/pull/37694#issuecomment-1229235723

   This does lead to two slf implementation being included with a hadoop-provided build (log4j 1 or reload4j with newer Hadoop versions). Not sure if there's a better way to handle this, or if Spark will always use it's slf implementation because that gets loaded first. Don't know the exact behavior when two implementations are provided.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org