geode-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (JIRA)" <>
Subject [jira] [Commented] (GEODE-1025) Remove compile dependency on spring data gemfire/geode
Date Fri, 08 Apr 2016 16:26:25 GMT


ASF subversion and git services commented on GEODE-1025:

Commit 22906e0b3d0926c09bb260bcd312863282b7a3bd in incubator-geode's branch refs/heads/feature/GEODE-17-2
from [~upthewaterspout]
[;h=22906e0 ]

GEODE-1025: Switching the latest SDG version and reanabling SDG test

Spring Data Gemfire (SDG) now implements a service provider required
to start the cache server with spring. Switching to point at the latest
SDG snapshot and renabling the test for starting with SDG.

> Remove compile dependency on spring data gemfire/geode
> ------------------------------------------------------
>                 Key: GEODE-1025
>                 URL:
>             Project: Geode
>          Issue Type: Improvement
>          Components: build
>            Reporter: Dan Smith
>            Assignee: Dan Smith
>             Fix For: 1.0.0-incubating.M2
> We have a compile dependency on spring data gemfire in the ServerLauncher class to support
launching a server using a spring data gemfire configuration instead of a cache.xml file.
> That's a good feature to have, but it should be part of spring data gemfire, not geode.
Because of this compile dependency on SDG, we currently have a circular dependency on SDG
-> Geode -> SDG. That makes it difficult to do things like rename packages in geode
(GEODE-37) because SDG is still using the old packages.
> Instead of directly depending on spring data gemfire, we should provide an extension
point that SDG can plug into to override the cache server startup behavior.
> The easiest place way to do this is just to use the java ServiceLoader mechanism. We
can provide an interface for a plugin that replaces startup behavior of the server. SDG can
provide an implementation of this interface.

This message was sent by Atlassian JIRA

View raw message