Based on the lack of “-1” votes, I have cherry-picked the change…
Thanks,
Mark
> On Nov 7, 2019, at 8:54 AM, Bruce Schuchardt wrote:
>
> The change passed in SSL benchmark testing and can be cherry-picked into the
> release branch. The Benchmark job as a whole failed due to perf degradation
+1 to include the fix
On Thu, Nov 7, 2019 at 8:54 AM Bruce Schuchardt
wrote:
> The change passed in SSL benchmark testing and can be cherry-picked into
> the release branch. The Benchmark job as a whole failed due to perf
> degradation with the Security Manager, but that's a different set of te
The change passed in SSL benchmark testing and can be cherry-picked into
the release branch. The Benchmark job as a whole failed due to perf
degradation with the Security Manager, but that's a different set of tests.
On 11/6/19 3:57 PM, Helena Bales wrote:
+1 to cherry-picking the fix.
The
+1 to cherry-picking the fix.
The sha hasn't made it to benchmarks yet due to an issue with CI losing
resource refs that were needed to keep it moving through the pipeline. The
next commit is still about an hour away from triggering benchmarks.
In my manual benchmarking of this change, I found tha
Any other votes? I have 2 people in favor.
Voting will close at noon.
Thanks,
Mark
> On Nov 6, 2019, at 8:00 AM, Bruce Schuchardt wrote:
>
> The fix for this problem is in the CI pipeline today:
> https://concourse.apachegeode-ci.info/teams/main/pipelines/apache-develop-main/jobs/Build/builds
The fix for this problem is in the CI pipeline today:
https://concourse.apachegeode-ci.info/teams/main/pipelines/apache-develop-main/jobs/Build/builds/1341
On 11/5/19 10:49 AM, Owen Nichols wrote:
+1 for bringing this fix to release/1.11.0 (after it has passed Benchmarks on
develop)
On Nov 5
+1 for bringing this fix to release/1.11.0 (after it has passed Benchmarks on
develop)
> On Nov 5, 2019, at 10:45 AM, Bruce Schuchardt wrote:
>
> The PR for GEODE-6661 introduced a problem in SSL communications that needs
> to be fixed. It changed SSL handshakes to use a temporary buffer that
