Dan- 2 things:
1) Users and customers (alike), in certain cases, do launch Geode Servers using Spring Boot, either with java -jar and a FAT JAR or other ways. 2) I have been saying for sometime now that I think it would be nice to be able to launch a server using a classpath configured with a Maven POM file, sort of like... gfsh> start server --name=X --maven-pom=/path/to/maven/pom.xml --mave-pom would set the server classpath accordingly and reliably. All too often, users/customers miss critical dependencies on the cluster servers' classpath, especially transitive dependencies, that are required by their application artifacts: custom CacheLoaders, Functions, etc. Food for thought. -John On Thu, Feb 28, 2019 at 10:22 AM Dan Smith <dsm...@pivotal.io> wrote: > Currently, geode servers just have a flat classpath with all of the > dependencies of all of the modules. Having the ability to add optional > modules sounds like a good feature, though. > > John - what you are describing applies to maven dependencies - and I do > think that we should isolate optional features into separate maven > dependencies like geode-lucene. But that doesn't help with servers launched > through gfsh start server unless we provide a way to configure which geode > modules are present on the server's classpath. > > -Dan > > On Thu, Feb 28, 2019 at 10:03 AM John Blum <jb...@pivotal.io> wrote: > > > Well, that just requires that you appropriately declare dependencies with > > the "optionality" and "scope" (e.g. "compile", "test", "provided", etc). > > > > Additionally, Geode modules could selectively pull in the required deps > as > > needed. For example, `geode-lucene` would only pull in the Apache Lucene > > dependencies if the `geode-lucene` module is used. > > > > No brainer. > > > > On Thu, Feb 28, 2019 at 9:47 AM Charlie Black <cbl...@pivotal.io> 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 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > Charlie Black | cbl...@pivotal.io > > > > > > > > > -- > > -John > > john.blum10101 (skype) > > > -- -John john.blum10101 (skype)
