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> on 2020/07/01 15:33:22 UTC

Re: [PROPOSAL] merge GEODE-8259 to support branches

+1
I reviewed this PR and, as Gester said, it's low risk.  If it fixes a problem someone is having let's backport it.

On 6/30/20, 3:51 PM, "Xiaojian Zhou" <zh...@gmail.com> wrote:

    Customer encountered a singlehop getAll failure due to
    SerializationException which is identified as socket error. The solution is
    to retry the getAll in this race condition (currently we did not retry).


    The fix is tested in both develop and support branches. The fix is
    conservative and very low risk.



    So it would be nice to bring to before 1.13.0 release.



    Regards

    Xiaojian Zhou

Re: [PROPOSAL] merge GEODE-8259 to support branches

Posted by Dave Barnes <db...@apache.org>.

OK, Gester, please merge.
Thanks,
Dave

On Wed, Jul 1, 2020 at 8:33 AM Bruce Schuchardt <br...@vmware.com> wrote:

> +1
> I reviewed this PR and, as Gester said, it's low risk.  If it fixes a
> problem someone is having let's backport it.
>
> On 6/30/20, 3:51 PM, "Xiaojian Zhou" <zh...@gmail.com> wrote:
>
>     Customer encountered a singlehop getAll failure due to
>     SerializationException which is identified as socket error. The
> solution is
>     to retry the getAll in this race condition (currently we did not
> retry).
>
>
>     The fix is tested in both develop and support branches. The fix is
>     conservative and very low risk.
>
>
>
>     So it would be nice to bring to before 1.13.0 release.
>
>
>
>     Regards
>
>     Xiaojian Zhou
>
>