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/06/16 22:24:04 UTC

[PROPOSAL] backport PR #5250 to support/1.13

This PR has been merged to develop.  It fixes a problem with the previous commit for GEODE-8144 that caused performance degradation when TLS is enabled between servers.  I have run perf tests and verified that it fixes the problem.

It’s a small change that makes a big difference…
https://github.com/apache/geode/pull/5250


RE: [PROPOSAL] backport PR #5250 to support/1.13

Posted by Dick Cavender <di...@vmware.com>.
+1

-----Original Message-----
From: Bruce Schuchardt <br...@vmware.com> 
Sent: Tuesday, June 16, 2020 3:24 PM
To: dev@geode.apache.org
Subject: [PROPOSAL] backport PR #5250 to support/1.13

This PR has been merged to develop.  It fixes a problem with the previous commit for GEODE-8144 that caused performance degradation when TLS is enabled between servers.  I have run perf tests and verified that it fixes the problem.

It’s a small change that makes a big difference…
https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5250&amp;data=02%7C01%7Cdickc%40vmware.com%7C3c5cb9fbd9274b032b8508d8124403ad%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C637279430617900387&amp;sdata=eT22ksaNTZIFI%2F14NrTkauiiDclGug37g0SEX7Uklek%3D&amp;reserved=0


Re: [PROPOSAL] backport PR #5250 to support/1.13

Posted by Dave Barnes <db...@apache.org>.
Looks good, Bruce. Please backport.


On Tue, Jun 16, 2020 at 4:01 PM Owen Nichols <on...@vmware.com> wrote:

> +1
>
> On 6/16/20, 3:54 PM, "Jianxia Chen" <jc...@apache.org> wrote:
>
>     +1
>
>     On Tue, Jun 16, 2020 at 3:24 PM Bruce Schuchardt <br...@vmware.com>
> wrote:
>
>     > This PR has been merged to develop.  It fixes a problem with the
> previous
>     > commit for GEODE-8144 that caused performance degradation when TLS is
>     > enabled between servers.  I have run perf tests and verified that it
> fixes
>     > the problem.
>     >
>     > It’s a small change that makes a big difference…
>     >
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5250&amp;data=02%7C01%7Conichols%40vmware.com%7Ce2ec5e282bb240a6406d08d812484504%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C637279448892893348&amp;sdata=1qOm9MNcOb0JbWE8PTtLJEicYXbO8QglFaPk18VhRss%3D&amp;reserved=0
>     >
>     >
>
>

Re: [PROPOSAL] backport PR #5250 to support/1.13

Posted by Owen Nichols <on...@vmware.com>.
+1

On 6/16/20, 3:54 PM, "Jianxia Chen" <jc...@apache.org> wrote:

    +1

    On Tue, Jun 16, 2020 at 3:24 PM Bruce Schuchardt <br...@vmware.com> wrote:

    > This PR has been merged to develop.  It fixes a problem with the previous
    > commit for GEODE-8144 that caused performance degradation when TLS is
    > enabled between servers.  I have run perf tests and verified that it fixes
    > the problem.
    >
    > It’s a small change that makes a big difference…
    > https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5250&amp;data=02%7C01%7Conichols%40vmware.com%7Ce2ec5e282bb240a6406d08d812484504%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C637279448892893348&amp;sdata=1qOm9MNcOb0JbWE8PTtLJEicYXbO8QglFaPk18VhRss%3D&amp;reserved=0
    >
    >


Re: [PROPOSAL] backport PR #5250 to support/1.13

Posted by Jianxia Chen <jc...@apache.org>.
+1

On Tue, Jun 16, 2020 at 3:24 PM Bruce Schuchardt <br...@vmware.com> wrote:

> This PR has been merged to develop.  It fixes a problem with the previous
> commit for GEODE-8144 that caused performance degradation when TLS is
> enabled between servers.  I have run perf tests and verified that it fixes
> the problem.
>
> It’s a small change that makes a big difference…
> https://github.com/apache/geode/pull/5250
>
>