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

Posted to issues@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2022/03/31 17:38:34 UTC

[GitHub] [bookkeeper] dlg99 commented on a change in pull request #3167: [branch-4.14] Replace Log4J with Reload4J

dlg99 commented on a change in pull request #3167:
URL: https://github.com/apache/bookkeeper/pull/3167#discussion_r839866094



##########
File path: pom.xml
##########
@@ -160,7 +160,7 @@
     <reflections.version>0.9.11</reflections.version>
     <rocksdb.version>6.29.4.1</rocksdb.version>
     <shrinkwrap.version>3.0.1</shrinkwrap.version>
-    <slf4j.version>1.7.32</slf4j.version>
+    <slf4j.version>1.7.36</slf4j.version>

Review comment:
       that got me confused for a sec, why there is no new dependency added.
   In case someone has the same question, this is because of 
   > The SLF4J project has released version 1.7.35 containing the slf4j-reload4j module which supports reload4j. Moreover, the "slf4j-log4j12" artifact contains a Maven relocation directive for the "slf4j-reload4j" artifact.
   
   https://reload4j.qos.ch/
   




-- 
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: issues-unsubscribe@bookkeeper.apache.org

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