gemmellr commented on PR #4540:
URL: 
https://github.com/apache/activemq-artemis/pull/4540#issuecomment-1631139595

   Presumably #4545 helps this PR not error as-is, since it uses a newer 
version of guava than the 27.0.1 that the old 5.2.0 curator did, one new enough 
to satisfy error-prones needs? It is however still using an old version of 
guava, 31.1-jre, so note we will still want the guava dependencyManagement 
restored to bring it back up to the 32.0.1-jre release it was at before this PR.
   
   In hindsight, removing the guava from the console war may not have broken 
the console in the assembly, since guava is still going to be brought in to the 
assembly via the quorum bits. It should still remain in the war though as the 
server itself doesnt bring it in with these changes, and the console can be/is 
used with the server without using the assembly.
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to