On Thu, 29 Oct 2020 15:11:09 GMT, Christoph Langer <clan...@openjdk.org> wrote:

> It seems that there exists a memory/performance regression that was 
> introduced with JDK-8210985: Update the default SSL session cache size to 
> 20480.
> 
> The idea to limit the maixmum SSL session cache size by itself is good. 
> Unfortunately, as per the current implementation of 
> sun.security.util.MemoryCache, it also modifies the initial size of the 
> LinkedHashMap that is backing the cache to initialize with more than the 
> maximum size.
> 
> I suggest to restore the original behavior that initializes with an 
> initialCapacity of 1 in most cases. That was true when before JDK-8210985, 
> the property javax.net.ssl.sessionCacheSize was not set at all.
> In case it was set, the initial size would have been like now, 
> (javax.net.ssl.sessionCacheSize / 0.75f) + 1, which still seems strange.

This pull request has now been integrated.

Changeset: 64feeab7
Author:    Christoph Langer <clan...@openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/64feeab7
Stats:     3 lines in 1 file changed: 0 ins; 1 del; 2 mod

8255603: Memory/Performance regression after JDK-8210985

Reviewed-by: simonis, xuelei, aph

-------------

PR: https://git.openjdk.java.net/jdk/pull/937

Reply via email to