Hi there Owen,

I apologize if it is not clear.

GEODE-7241 is a regression where the published artifact "geode-web" and "geode-web-api" were published as jars and not wars.

NOW, why is it critical for 1.9.x... Well, the answer is fairly short. In Spring Data Geode, there is an ability to start/bootstrap a server using Spring Data Geode(SDG) only. This feature can be used for testing as well as starting a Server using SDG. In order to start the server using SDG, all that is required is to add the geode-web + geode-web-api artifacts onto the classpath as a maven/gradle dependency. There is no more requirement to download the project and reference libs or using GFSH to start the server.

WHEN the extension went from war -> jar, this functionality was broken. The latest version of SDG (2.2.x) will be based on Geode 1.9. and not 1.10+ which means that SDG will not gain that functionality. SDG would have to wait for its next release 2.3.x, which is still quite some way off.

Usually this is not a big thing, but given Spring's prescribed manner of handling versions of dependent libraries, the version of Geode cannot be changed and only patch versions are allowed. So, in order to address the regression, a patch to 1.9.x is requested.

Hope that this explains it a little better.

--Udo

On 10/7/19 3:50 PM, Owen Nichols wrote:
I don’t yet have a clear understanding of what makes these critical,
especially GEODE-7241. Can you elaborate, including:
* Are these fixes already in 1.10?  If not, would a 1.10.1 patch be
required as well?
* What is the impact of not including each of these fixes?  Is there a
workaround?

On Fri, Oct 4, 2019 at 10:44 AM Jens Deppe <jde...@pivotal.io> wrote:

I'd like to propose adding these two fixes to release/1.9.2

GEODE-7261 ensures that the Admin REST service can start when Spring Boot
Data Geode is used to launch a server.

GEODE-7241 publishes our various war artifacts to maven. This ensures that,
in the context of starting a SBDG server, the necessary REST wars can be
made available in order to start the required REST services.

Thanks

--Jens

Reply via email to