I wonder if we should be looking at shaded jars for the extensions.
--Udo
On 2/28/19 09:47, Charlie Black wrote:
Hopefully, we are thinking about classpath of the server and lazily adding
these jars only when a feature is turned on.
On Thu, Feb 28, 2019 at 9:45 AM Dan Smith <dsm...@pivotal.io> wrote:
I see that geo, grumpy-core, and commons math came from adding geospatial
support to redis -
https://github.com/apache/geode/commit/7bf02251fd047cb1cf575c01b80a9807108618da
-Dan
On Thu, Feb 28, 2019 at 9:41 AM Anthony Baker <aba...@pivotal.io> wrote:
Looks a number of the new dependencies came in transitively with the
guava
version bump.
On Feb 27, 2019, at 5:32 PM, Anthony Baker <aba...@pivotal.io> wrote:
I was reviewing the release branch and noticed a number of new
dependencies have been added since the last release. When you add a new
dependency, please review and follow the project license guide [1]. In
particular, update the LICENSE file in geode-assembly/src/main/dist
depending on the license type.
Currently we need to update the LICENSE file with the additional
MIT/BSD/CDDL dependencies. We may also need to update NOTICE files.
There’s also a version conflict with multiple versions of Jackson in use
(2.9.6 / 2.9.8).
@Sai - these need to be fixed on release/1.9.0
Here’s the list of additions:
animal-sniffer-annotations-1.17.jar
checker-qual-2.5.2.jar
commons-math3-3.2.jar
error_prone_annotations-2.2.0.jar
failureaccess-1.0.jar
geo-0.7.1.jar
grumpy-core-0.2.2.jar
istack-commons-runtime-2.2.jar
j2objc-annotations-1.1.jar
javax.activation-1.2.0.jar
javax.activation-api-1.2.0.jar
jsr305-3.0.2.jar
listenablefuture-9999.0-empty-to-avoid-conflict-with-guava.jar
Removed:
activation-1.1.1
jaxb-core-2.2.11.jar
Anthony
[1]
https://cwiki.apache.org/confluence/display/GEODE/License+Guide+for+Contributors
<
https://cwiki.apache.org/confluence/display/GEODE/License+Guide+for+Contributors
