egor-ryashin commented on issue #6942: URL: https://github.com/apache/druid/issues/6942#issuecomment-845390864
I've been recently reviewing the PR https://github.com/apache/druid/pull/10964 Seems good, but we still have other libs dependent on the old Guava, ie: ``` mvn dependency:tree ... [INFO] +- com.google.inject:guice:jar:4.1.0:provided [INFO] | +- aopalliance:aopalliance:jar:1.0:provided [INFO] | \- com.google.guava:guava:jar:16.0.1:compile ``` I've actually been blocked by the issue while developing an extension with Google libraries that internally used newer Guava. And after merging this https://github.com/apache/druid/pull/10964 the Druid will still be packaged with Guava jar of version 16. I suppose https://github.com/apache/druid/pull/10964 is the first step and there's a plan to upgrade Guice and other libs dependent on Guava 16? (BTW, right now to load an extension into Druid I use Java Reflection along with a different classloader which is very constraining.) -- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
