Re: [PROPOSAL]: Include GEODE-7789 in Release 1.12.0

2020-02-13 Thread Ernest Burghardt
There appears to be consensus that this is a critical fix. The following commit has been brought into release/1.12 as the critical fix for GEODE-7789: git cherry-pick -x 71e156a55228d89efafd048e1533debba606c064 GEODE- has been marked as 'resolved in' 1.12. Regards EB On Thu, Feb 13, 2020

Re: [PROPOSAL]: Include GEODE-7789 in Release 1.12.0

2020-02-13 Thread Donal Evans
+1 On Thu, Feb 13, 2020, 02:09 Ju@N wrote: > Hello devs, > > I'd like to include the fix for GEODE-7789 [1] in release 1.12.0. > The change prevents a performance degradation introduced in Geode 1.11, it > basically shows up on clusters where there's at least one client with > subscription enabl

Re: [PROPOSAL]: Include GEODE-7789 in Release 1.12.0

2020-02-13 Thread Jianxia Chen
+1 On Thu, Feb 13, 2020 at 7:11 AM Udo Kohlmeyer wrote: > +1 > > On 2/13/20 6:49 AM, Jinmei Liao wrote: > > +1 > > > > On Thu, Feb 13, 2020, 6:47 AM Owen Nichols wrote: > > > >> +1 > >> > >> On Thu, Feb 13, 2020 at 3:05 AM Nabarun Nag wrote: > >> > >>> +1 > >>> > >>> On Thu, Feb 13, 2020 at 2:

Re: [PROPOSAL]: Include GEODE-7789 in Release 1.12.0

2020-02-13 Thread Udo Kohlmeyer
+1 On 2/13/20 6:49 AM, Jinmei Liao wrote: +1 On Thu, Feb 13, 2020, 6:47 AM Owen Nichols wrote: +1 On Thu, Feb 13, 2020 at 3:05 AM Nabarun Nag wrote: +1 On Thu, Feb 13, 2020 at 2:09 AM Ju@N wrote: Hello devs, I'd like to include the fix for GEODE-7789 [1] in release 1.12.0. The chang

Re: [PROPOSAL]: Include GEODE-7789 in Release 1.12.0

2020-02-13 Thread Jinmei Liao
+1 On Thu, Feb 13, 2020, 6:47 AM Owen Nichols wrote: > +1 > > On Thu, Feb 13, 2020 at 3:05 AM Nabarun Nag wrote: > > > +1 > > > > On Thu, Feb 13, 2020 at 2:09 AM Ju@N wrote: > > > > > Hello devs, > > > > > > I'd like to include the fix for GEODE-7789 [1] in release 1.12.0. > > > The change pre

Re: [PROPOSAL]: Include GEODE-7789 in Release 1.12.0

2020-02-13 Thread Owen Nichols
+1 On Thu, Feb 13, 2020 at 3:05 AM Nabarun Nag wrote: > +1 > > On Thu, Feb 13, 2020 at 2:09 AM Ju@N wrote: > > > Hello devs, > > > > I'd like to include the fix for GEODE-7789 [1] in release 1.12.0. > > The change prevents a performance degradation introduced in Geode 1.11, > it > > basically s

Re: [PROPOSAL]: Include GEODE-7789 in Release 1.12.0

2020-02-13 Thread Nabarun Nag
+1 On Thu, Feb 13, 2020 at 2:09 AM Ju@N wrote: > Hello devs, > > I'd like to include the fix for GEODE-7789 [1] in release 1.12.0. > The change prevents a performance degradation introduced in Geode 1.11, it > basically shows up on clusters where there's at least one client with > subscription e

[PROPOSAL]: Include GEODE-7789 in Release 1.12.0

2020-02-13 Thread Ju@N
Hello devs, I'd like to include the fix for GEODE-7789 [1] in release 1.12.0. The change prevents a performance degradation introduced in Geode 1.11, it basically shows up on clusters where there's at least one client with subscription enabled. The SHA is 71e156a55228d89efafd048e1533debba606c064 [