[GitHub] [activemq] cshannon commented on pull request #766: [AMQ-8472] Fix reload4j scope in activemq-partition module

[GitBox <gi...@apache.org>]

cshannon commented on pull request #766:
URL: https://github.com/apache/activemq/pull/766#issuecomment-1036218508


   @jbonofre  - So @gemmellr  and I have been talking and this might have already been correct. The issue is apparently the Zookeeper dependencies (and linkedin) have a hard dependency on log4j so it sounds like if someone wants to use this module they actually do need a compile time dependency on reload4j so that it gets pulled in and doesn't break. Robbie can comment as well but I think maybe we just leave it as it already was? It's gone in 5.17.0 anyways
   
   Even if we don't make this change at least we can pull in the license file changes (updated years) and re-cut the release since it got cancelled already.


-- 
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: gitbox-unsubscribe@activemq.apache.org

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