You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geode.apache.org by Bruce Schuchardt <br...@vmware.com.INVALID> on 2020/02/28 19:27:42 UTC
[PROPOSAL] include GEODE-7829 in 1.12
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.
Re: [PROPOSAL] include GEODE-7829 in 1.12
Posted by Ernest Burghardt <eb...@pivotal.io>.
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 at 1:20 PM Donal Evans <do...@pivotal.io> wrote:
> +1
>
> On Fri, Feb 28, 2020 at 1:15 PM Ernest Burghardt <eb...@pivotal.io>
> 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 <on...@pivotal.io>
> 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 Feb 28, 2020, at 1:06 PM, Ernest Burghardt <ec...@apache.org>
> > > wrote:
> > > >
> > > > bumping this out to DevList as something went wrong in the mail
> > system...
> > > >
> > > > On 2020/02/28 19:27:42, Bruce Schuchardt <br...@vmware.com.INVALID>
> > > 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-threshold determines how long we wait for a response
> to a
> > > message, such as replication of a put(), before taking some action.
> > > >>
> > >
> > >
> >
>
Re: [PROPOSAL] include GEODE-7829 in 1.12
Posted by Donal Evans <do...@pivotal.io>.
+1
On Fri, Feb 28, 2020 at 1:15 PM Ernest Burghardt <eb...@pivotal.io>
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 <on...@pivotal.io> 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 Feb 28, 2020, at 1:06 PM, Ernest Burghardt <ec...@apache.org>
> > wrote:
> > >
> > > bumping this out to DevList as something went wrong in the mail
> system...
> > >
> > > On 2020/02/28 19:27:42, Bruce Schuchardt <br...@vmware.com.INVALID>
> > 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-threshold determines how long we wait for a response to a
> > message, such as replication of a put(), before taking some action.
> > >>
> >
> >
>
Re: [PROPOSAL] include GEODE-7829 in 1.12
Posted by Ernest Burghardt <eb...@pivotal.io>.
yes, it was just a transposition/typo when the defaults were
refactored/moved
On Fri, Feb 28, 2020 at 1:14 PM Owen Nichols <on...@pivotal.io> 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 Feb 28, 2020, at 1:06 PM, Ernest Burghardt <ec...@apache.org>
> wrote:
> >
> > bumping this out to DevList as something went wrong in the mail system...
> >
> > On 2020/02/28 19:27:42, Bruce Schuchardt <br...@vmware.com.INVALID>
> 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-threshold determines how long we wait for a response to a
> message, such as replication of a put(), before taking some action.
> >>
>
>
Re: [PROPOSAL] include GEODE-7829 in 1.12
Posted by Owen Nichols <on...@pivotal.io>.
+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 <ec...@apache.org> wrote:
>
> bumping this out to DevList as something went wrong in the mail system...
>
> On 2020/02/28 19:27:42, Bruce Schuchardt <br...@vmware.com.INVALID> 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-threshold determines how long we wait for a response to a message, such as replication of a put(), before taking some action.
>>
Re: [PROPOSAL] include GEODE-7829 in 1.12
Posted by Ernest Burghardt <ec...@apache.org>.
bumping this out to DevList as something went wrong in the mail system...
On 2020/02/28 19:27:42, Bruce Schuchardt <br...@vmware.com.INVALID> 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-threshold determines how long we wait for a response to a message, such as replication of a put(), before taking some action.
>