Re: [DISCUSS] Add GEODE-7261 and GEODE-7241 to release/1.9.2

2019-10-14 Thread Jens Deppe
Thank you for the approvals. I will include these changes in 1.9.2 --Jens On Mon, Oct 14, 2019 at 11:21 AM John Blum wrote: > +1 > > On Mon, Oct 14, 2019 at 11:19 AM Udo Kohlmeyer wrote: > > > +1 to including both. > > > > On 10/14/19 10:52 AM, Dick Cavender wrote: > > > +1 for both fixes and

Re: [DISCUSS] Add GEODE-7261 and GEODE-7241 to release/1.9.2

2019-10-14 Thread John Blum
+1 On Mon, Oct 14, 2019 at 11:19 AM Udo Kohlmeyer wrote: > +1 to including both. > > On 10/14/19 10:52 AM, Dick Cavender wrote: > > +1 for both fixes and the original list > > > > > > On Mon, Oct 7, 2019 at 5:00 PM Owen Nichols wrote: > > > >> Sounds like a big win for convenience, and clearly

Re: [DISCUSS] Add GEODE-7261 and GEODE-7241 to release/1.9.2

2019-10-14 Thread Udo Kohlmeyer
+1 to including both. On 10/14/19 10:52 AM, Dick Cavender wrote: +1 for both fixes and the original list On Mon, Oct 7, 2019 at 5:00 PM Owen Nichols wrote: Sounds like a big win for convenience, and clearly a regression relative to the last release of Geode that SBDG picked up (1.6).

Re: [DISCUSS] Add GEODE-7261 and GEODE-7241 to release/1.9.2

2019-10-14 Thread Dick Cavender
+1 for both fixes and the original list On Mon, Oct 7, 2019 at 5:00 PM Owen Nichols wrote: > Sounds like a big win for convenience, and clearly a regression relative > to the last release of Geode that SBDG picked up (1.6). Thanks for > clarifying what is at stake. > > +1 for including both

Re: [DISCUSS] Add GEODE-7261 and GEODE-7241 to release/1.9.2

2019-10-07 Thread Owen Nichols
Sounds like a big win for convenience, and clearly a regression relative to the last release of Geode that SBDG picked up (1.6). Thanks for clarifying what is at stake. +1 for including both fixes > On Oct 7, 2019, at 4:50 PM, Udo Kohlmeyer wrote: > > Hi there Owen, > > I apologize if it

Re: [DISCUSS] Add GEODE-7261 and GEODE-7241 to release/1.9.2

2019-10-07 Thread Udo Kohlmeyer
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

Re: [DISCUSS] Add GEODE-7261 and GEODE-7241 to release/1.9.2

2019-10-07 Thread Owen Nichols
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

Re: [DISCUSS] Add GEODE-7261 and GEODE-7241 to release/1.9.2

2019-10-07 Thread Jens Deppe
Bump - reminder for folks to consider these two changes be included in Geode 1.9.2 Thanks --Jens On Fri, Oct 4, 2019 at 12:57 PM Juan José Ramos wrote: > +1 > > On Fri, Oct 4, 2019 at 6:44 PM Jens Deppe wrote: > > > I'd like to propose adding these two fixes to release/1.9.2 > > > >

Re: [DISCUSS] Add GEODE-7261 and GEODE-7241 to release/1.9.2

2019-10-04 Thread Juan José Ramos
+1 On Fri, Oct 4, 2019 at 6:44 PM Jens Deppe 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.

[DISCUSS] Add GEODE-7261 and GEODE-7241 to release/1.9.2

2019-10-04 Thread Jens Deppe
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