[GitHub] [hadoop] steveloughran commented on pull request #5315: HADOOP-18206 Cleanup the commons-logging references and restrict its usage in future
steveloughran commented on PR #5315: URL: https://github.com/apache/hadoop/pull/5315#issuecomment-1419375416 just noticed that http-components pulls in commons logging so things which depend on that still need to export it. question then becomes: do we want to explicitly manage the dependency there? ``` [info] | | +-org.apache.hadoop:hadoop-azure:3.3.5 [info] | | | +-com.microsoft.azure:azure-storage:7.0.1 [info] | | | | +-com.microsoft.azure:azure-keyvault-core:1.0.0 [info] | | | | +-org.slf4j:slf4j-api:1.7.12 (evicted by: 2.0.6) [info] | | | | +-org.slf4j:slf4j-api:2.0.6 [info] | | | | [info] | | | +-org.apache.hadoop.thirdparty:hadoop-shaded-guava:1.1.1 [info] | | | +-org.apache.httpcomponents:httpclient:4.5.13 (evicted by: 4.5.14) [info] | | | +-org.apache.httpcomponents:httpclient:4.5.14 [info] | | | | +-commons-codec:commons-codec:1.11 (evicted by: 1.15) [info] | | | | +-commons-codec:commons-codec:1.15 [info] | | | | +-commons-logging:commons-logging:1.2 [info] | | | | +-org.apache.httpcomponents:httpcore:4.4.16 ``` -- 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: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[GitHub] [hadoop] steveloughran commented on pull request #5315: HADOOP-18206 Cleanup the commons-logging references and restrict its usage in future
steveloughran commented on PR #5315: URL: https://github.com/apache/hadoop/pull/5315#issuecomment-1408635376 so those production classes I am worrying about are already explicitly requiring log4j on the CP? OK, that's not as bad as I feared...I was worrying that the code was going from commons-logging wrappers of logj to raw log4j, which is a problem -- 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: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org