[ https://issues.apache.org/jira/browse/GEODE-10151?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17514778#comment-17514778 ]
ASF subversion and git services commented on GEODE-10151: --------------------------------------------------------- Commit 3fd631a22f1a7f319cb38f1039827e7cbdd6e528 in geode's branch refs/heads/develop from Dan Smith [ https://gitbox.apache.org/repos/asf?p=geode.git;h=3fd631a ] GEODE-10151: Make geode-for-redis optional for a geode server Removing the geode-for-redis project and it's dependencies from the default classpath of a geode server. Before this change, geode-for-redis was installed in the lib directory of the server along with it's dependencies, and it was not easy to run without it. Creating a distribution for geode-for-redis that can be added to the classpath of a server if the user chooses. This distribution is currently being built in the geode-for-redis/build/distributions directory, as well as placed as a zip file in the tools/Modules/ directory of the overall geode distribution. This distribution only contains dependencies that are not already on the classpath of a geode server. To generate the redis distribution run ./gradlew geode-for-redis:installDist or for the whole geode distribution, including the redis zip file run ./gradlew geode-assembly:installDist See the gradle documentation on the distribution plugin for more details. The redis tests that previously required a GEODE_HOME to run now also require a GEODE_FOR_REDIS_HOME directory that points into an installed distribution of geode-for-redis. The tests add the geode-for-redis jars into the classpath using this environment variable. When running tests through gradle we set the GEODE_FOR_REDIS_HOME environment variable automatically. > Make geode-for-redis an optional dependency of a geode server > ------------------------------------------------------------- > > Key: GEODE-10151 > URL: https://issues.apache.org/jira/browse/GEODE-10151 > Project: Geode > Issue Type: Improvement > Components: redis > Reporter: Dan Smith > Assignee: Dan Smith > Priority: Major > Labels: pull-request-available > Fix For: 1.15.0 > > > The geode-for-redis module and its related dependencies are on the classpath > of the server by default when starting a server with gfsh start server or > using the geode dependencies jar. > We should make the module an optional dependencies, like some of our other > extensions (for example, the tomcat http session modules). This would allow a > user to choose if they want redis on the classpath or not. -- This message was sent by Atlassian Jira (v8.20.1#820001)