----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44865/ -----------------------------------------------------------
Review request for geode, Anthony Baker, Jason Huynh, Jens Deppe, and Mark Bretl. Repository: geode Description ------- This is the first step in cleaning up the pom. Enabling transitive dependencies allows us to reduce the number of dependencies we add, as well as making it easier to change the dependencies because we don't have few dependencies in our build file that aren't direct dependencies of geode. Adding excludes and forcing some versions to clean up the list of jars we are pulling in to match what it was before I enabled transitive dependencies. Some of these excludes and all of the forced versions look suspect. They need to be reviewed to see if we really need them. Adding support to mark jars as optional by setting an ext.optional property in the dependency declaration. Adding the optional property to jars in the geode-core project that are not required for geode to work. Some of these optional dependencies will go away if we can move things like the REST API, gfsh, and management classes into another subproject. Diffs ----- build.gradle 85e5c3564e0a347f4442d064f800f45822d53302 extensions/geode-modules-hibernate/build.gradle PRE-CREATION extensions/geode-modules-tomcat7/build.gradle PRE-CREATION extensions/geode-modules/build.gradle 1f9bff8d7d37700627ca1b6e8aa74827d59b76d4 geode-assembly/build.gradle 5930f13c5307e0cd70396b019263176a377d415a geode-core/build.gradle 041dc07c860c008f117d37969ee688375c2a348d geode-core/src/main/java/com/gemstone/gemfire/cache/query/internal/CumulativeNonDistinctResults.java 3735956f3f96f1d7a4fe4a7d2d9119947154a17f geode-junit/build.gradle PRE-CREATION geode-lucene/build.gradle 6218e55c73f771a46a74ae31606ce1ace5b2008f geode-pulse/build.gradle PRE-CREATION geode-rebalancer/build.gradle e3ba6abb47fee30f9b3fca978aac51c8b4ee95d9 geode-web-api/build.gradle 940fddd7b653ca3fc49916b6ff463c89f6e2f4aa geode-web/build.gradle 596590b1c96e016e2dd927a21dc50d2e373b62b2 gradle/dependency-resolution.gradle PRE-CREATION gradle/java.gradle PRE-CREATION gradle/publish.gradle PRE-CREATION Diff: https://reviews.apache.org/r/44865/diff/ Testing ------- Thanks, Dan Smith
