vdiravka commented on a change in pull request #1682: DRILL-7081: Upgrade
GlassFish Jersey and Javax Servlet dependecies
URL: https://github.com/apache/drill/pull/1682#discussion_r263505869
##########
File path: exec/jdbc-all/pom.xml
##########
@@ -576,7 +576,7 @@
This is likely due to you adding new dependencies to
a java-exec and not updating the excludes in this module. This is important as
it minimizes the size of the dependency of Drill application users.
</message>
- <maxsize>37500000</maxsize>
+ <maxsize>38600000</maxsize>
Review comment:
I forgot to check jdbc-all exclusions.
So `jetty` was fully excluded before updates. And almost all `Glassfish
Jersey` dependencies were excluded. But I had added new one without exclusion -
`org.glassfish.jersey.inject:jersey-hk2`.
For now it is excluded and jar file size can be untouched.
----------------------------------------------------------------
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]
With regards,
Apache Git Services
