-
Notifications
You must be signed in to change notification settings - Fork 3.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
remove use log of log4j v1 #15984
remove use log of log4j v1 #15984
Conversation
@FrankChen021 when you have time, could you help to review the code There is one integration test failed(integration-index-tests-indexer (kafka-index)), which is not related to the modification from the log
|
@AlbericByte Can you please test a druid build with HDFS storage and see if this patch does not break logging. |
@cryptoe Thanks for your time and suggestion.
|
I think @cryptoe meant to ask if the change is manually verified with HDFS as deep storage. |
@abhishekagarwal87 and @cryptoe
|
Fixes #12425.
Description
log4j v1 is end of life and full of security issues
indexing-hadoop/pom.xml
extensions-core/hdfs-storage/pom.xml
remove log4j v1 dependences
Fixed the bug to remove log4j v1
Release note
remove log4j v1 dependences
Key changed/added classes in this PR
pom.xml
This PR has: