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). Thank

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 fi

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 is

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 star

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 Fri,

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 > > > > GEODE-7261

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. Th