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

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/02/17 12:40:35 UTC

[GitHub] [pulsar] lhotari edited a comment on pull request #14300: Remove --illegal-access errors resulting from Google Guice - Pulsar IO, Offloaders and Pulsar SQL - Bump Guice to 5.1.0

lhotari edited a comment on pull request #14300:
URL: https://github.com/apache/pulsar/pull/14300#issuecomment-1042908600


   @nicoloboschi Perhaps it would make sense to use `guice-bom` and import that in dependencyManagement ? That would ensure that Guice versions aren't mixed for possible other Guice sub-components. WDYT? I merged this one already so it would need a new PR if you find it useful.


-- 
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: commits-unsubscribe@pulsar.apache.org

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