Re: [PR] [WIP][Spark 44646] Reduce usage of log4j core [spark]
github-actions[bot] closed pull request #45001: [WIP][Spark 44646] Reduce usage of log4j core URL: https://github.com/apache/spark/pull/45001 -- 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-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
Re: [PR] [WIP][Spark 44646] Reduce usage of log4j core [spark]
github-actions[bot] commented on PR #45001: URL: https://github.com/apache/spark/pull/45001#issuecomment-2113687236 We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable. If you'd like to revive this PR, please reopen it and ask a committer to remove the Stale tag! -- 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-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[PR] [WIP][Spark 44646] Reduce usage of log4j core [spark]
mucharafal opened a new pull request, #45001: URL: https://github.com/apache/spark/pull/45001 ### What changes were proposed in this pull request? Reducing usage of Log4j2 core classes, and moving this usage to isolated classes which won't be loaded unless log4j2 is on classpath. For now Spark strongly needs `log4j-core` library, what may conflicts with other libraries. ### Why are the changes needed? To let user decide which framework for logging he/she can use. ### Does this PR introduce _any_ user-facing change? Seems that no, but please verify. ### How was this patch tested? I tested builded `spark-common-utils` package locally against my test project. ### Was this patch authored or co-authored using generative AI tooling? No. -- 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-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
