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

Posted to jira@kafka.apache.org by GitBox <gi...@apache.org> on 2022/03/29 02:55:53 UTC

[GitHub] [kafka] showuon commented on pull request #11743: KAFKA-13660: Switch log4j12 to reload4j

showuon commented on pull request #11743:
URL: https://github.com/apache/kafka/pull/11743#issuecomment-1081347153


   > > @showuon I only found the log4j12 dependencies in the following projects:
   > 
   > > streams:upgrade-system-tests-0100
   > > streams:upgrade-system-tests-0101
   > 
   > > I guess they are not included in the release.
   > 
   > The log4j12 dependency is pulled into the project by the Streams library in version 0.10.0 and 0.10.1 that is used. These two project are only used for upgrade system tests. Thus, I think that is not an issue.
   
   @cadonna , agree. Then it LGTM now, except your PR is much more simpler than this one. Either one is good to me. Thanks.


-- 
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: jira-unsubscribe@kafka.apache.org

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