Re: [PROPOSAL] include GEODE-7829 in 1.12

2020-02-28 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-7829: git cherry-pick -x dfa3326e674278f38673c9508a13af752ae90e28 GEODE-7829 has been marked as 'resolved in' 1.12. Regards EB On Fri, Feb 28, 2020

Re: [PROPOSAL] include GEODE-7829 in 1.12

2020-02-28 Thread Donal Evans
+1 On Fri, Feb 28, 2020 at 1:15 PM Ernest Burghardt wrote: > yes, it was just a transposition/typo when the defaults were > refactored/moved > > On Fri, Feb 28, 2020 at 1:14 PM Owen Nichols wrote: > > > +1 for bringing this fix to 1.12 (assuming the change was unintentional?) > > > > Any ideas

Re: [PROPOSAL] include GEODE-7829 in 1.12

2020-02-28 Thread Ernest Burghardt
yes, it was just a transposition/typo when the defaults were refactored/moved On Fri, Feb 28, 2020 at 1:14 PM Owen Nichols wrote: > +1 for bringing this fix to 1.12 (assuming the change was unintentional?) > > Any ideas how much performance improvement this is anticipated to restore? > > > On Fe

Re: [PROPOSAL] include GEODE-7829 in 1.12

2020-02-28 Thread Owen Nichols
+1 for bringing this fix to 1.12 (assuming the change was unintentional?) Any ideas how much performance improvement this is anticipated to restore? > On Feb 28, 2020, at 1:06 PM, Ernest Burghardt wrote: > > bumping this out to DevList as something went wrong in the mail system... > > On 2020/

[PROPOSAL] include GEODE-7829 in 1.12

2020-02-28 Thread Ernest Burghardt
Hello devs, I'd like to include the fix for GEODE-782 [1] in release 1.12.0. During a refactor the default ack-wait-threshold was changed from 15 to 51. This will affect any deployment that doesn’t set its own threshold. The ack-wait-threshold determines how long we wait for a response to a me

Re: [PROPOSAL] include GEODE-7829 in 1.12

2020-02-28 Thread Ernest Burghardt
bumping this out to DevList as something went wrong in the mail system... On 2020/02/28 19:27:42, Bruce Schuchardt wrote: > During a refactor the default ack-wait-threshold was changed from 15 to 51. > This will affect any deployment that doesn’t set its own threshold. > > The ack-wait-thresh

[PROPOSAL] include GEODE-7829 in 1.12

2020-02-28 Thread Bruce Schuchardt
During a refactor the default ack-wait-threshold was changed from 15 to 51. This will affect any deployment that doesn’t set its own threshold. The ack-wait-threshold determines how long we wait for a response to a message, such as replication of a put(), before taking some action.