On Tue, 23 Apr 2024 09:05:25 GMT, Oli Gillespie wrote:
> > Please don't integrate until a Reviewer from the Security Group has
> > reviewed this. Thanks.
>
> @seanjmullan are you okay for me to integrate now?
Yes, @ascarpino has reviewed it and he is a Reviewer in the Security Group.
On Fri, 12 Apr 2024 16:19:56 GMT, Sean Mullan wrote:
> Please don't integrate until a Reviewer from the Security Group has reviewed
> this. Thanks.
@seanjmullan are you okay for me to integrate now?
-
PR Comment: https://git.openjdk.org/jdk/pull/18763#issuecomment-2071793073
On Fri, 12 Apr 2024 15:34:01 GMT, Oli Gillespie wrote:
> Increase buffer size in CipherInputStream from 512 bytes to 8192 bytes.
>
> I have seen applications where this small buffer size significantly reduces
> throughput, and I've even seen applications which use reflection to modify
> the bu
On Fri, 19 Apr 2024 09:09:56 GMT, Oli Gillespie wrote:
> Thanks :) - created release note at
> https://bugs.openjdk.org/browse/JDK-8330632, please review if you can.
Looks fine. You can mark it as Resolved/Delivered after the fix is integrated.
-
PR Comment: https://git.openjdk.or
On Fri, 12 Apr 2024 15:34:01 GMT, Oli Gillespie wrote:
> Increase buffer size in CipherInputStream from 512 bytes to 8192 bytes.
>
> I have seen applications where this small buffer size significantly reduces
> throughput, and I've even seen applications which use reflection to modify
> the bu
On Fri, 12 Apr 2024 15:34:01 GMT, Oli Gillespie wrote:
> Increase buffer size in CipherInputStream from 512 bytes to 8192 bytes.
>
> I have seen applications where this small buffer size significantly reduces
> throughput, and I've even seen applications which use reflection to modify
> the bu
On Sat, 13 Apr 2024 01:25:57 GMT, David Schlosnagle wrote:
> Should the buffer size be a configurable constructor parameter? This would
> allow consumers to adjust the buffer size to match their tradeoffs of memory
> vs throughput based on workload & available resources, and provide a bit of
>
On Fri, 12 Apr 2024 15:34:01 GMT, Oli Gillespie wrote:
> Increase buffer size in CipherInputStream from 512 bytes to 8192 bytes.
>
> I have seen applications where this small buffer size significantly reduces
> throughput, and I've even seen applications which use reflection to modify
> the bu
On Mon, 15 Apr 2024 19:34:18 GMT, Anthony Scarpino
wrote:
>> Can you provide memory usage difference between the current and your
>> suggested change with `-prof gc`? With many of these situations, it's a
>> balance between memory usage and performance.
>
>> @ascarpino here's the `-prof gc` r
On Fri, 12 Apr 2024 16:49:49 GMT, Anthony Scarpino
wrote:
>> Increase buffer size in CipherInputStream from 512 bytes to 8192 bytes.
>>
>> I have seen applications where this small buffer size significantly reduces
>> throughput, and I've even seen applications which use reflection to modify
On Fri, 12 Apr 2024 15:34:01 GMT, Oli Gillespie wrote:
> Increase buffer size in CipherInputStream from 512 bytes to 8192 bytes.
>
> I have seen applications where this small buffer size significantly reduces
> throughput, and I've even seen applications which use reflection to modify
> the bu
On Fri, 12 Apr 2024 16:49:49 GMT, Anthony Scarpino
wrote:
>> Increase buffer size in CipherInputStream from 512 bytes to 8192 bytes.
>>
>> I have seen applications where this small buffer size significantly reduces
>> throughput, and I've even seen applications which use reflection to modify
On Fri, 12 Apr 2024 15:34:01 GMT, Oli Gillespie wrote:
> Increase buffer size in CipherInputStream from 512 bytes to 8192 bytes.
>
> I have seen applications where this small buffer size significantly reduces
> throughput, and I've even seen applications which use reflection to modify
> the bu
On Fri, 12 Apr 2024 15:34:01 GMT, Oli Gillespie wrote:
> Increase buffer size in CipherInputStream from 512 bytes to 8192 bytes.
>
> I have seen applications where this small buffer size significantly reduces
> throughput, and I've even seen applications which use reflection to modify
> the bu
On Fri, 12 Apr 2024 15:34:01 GMT, Oli Gillespie wrote:
> Increase buffer size in CipherInputStream from 512 bytes to 8192 bytes.
>
> I have seen applications where this small buffer size significantly reduces
> throughput, and I've even seen applications which use reflection to modify
> the bu
Increase buffer size in CipherInputStream from 512 bytes to 8192 bytes.
I have seen applications where this small buffer size significantly reduces
throughput, and I've even seen applications which use reflection to modify the
buffer size to work around the issue.
Using the existing `AESGCMCiph
16 matches
Mail list logo
