Hi Sean,

Are you OK if we do it later? I'm waiting for the @systemProperty tag, proposed within JDK-5076751. I will file a bug to use the tag for more cleanup.

Thanks,
Xuelei

On 11/15/2018 11:55 AM, Sean Mullan wrote:
This is a good opportunity to document the javax.net.ssl.sessionCacheSize system property in the SSLSessionContext API (and use the new @systemProperty tag) in an @implNote, for example:

     /**
      * Returns the size of the cache used for storing
      * <code>SSLSession</code> objects grouped under this
      * <code>SSLSessionContext</code>.
      *
      * @implNote The JDK implementation returns the cache size as set by
      * the {@code setSessionCacheSize method}, or if not set, the value
      * of the {@systemProperty javax.net.ssl.sessionCacheSize} system
      * property. If neither is set, it returns a default value of 20480.
      *
     * @return size of the session cache; zero means there is no size limit.
      * @see #setSessionCacheSize
      */
     public int getSessionCacheSize();

On 11/14/18 11:59 AM, Xuelei Fan wrote:
Hi,

Please review this simple update:
     http://cr.openjdk.java.net/~xuelei/8210985/webrev.00/

The default value for the maximum number of entries in the SSL session cache (SSLSessionContext.getSessionCacheSize()) is infinite now.  In the request, the default value is updated to 20480 for performance consideration.

For the detailed behavior update, please refer to CSR:
     https://bugs.openjdk.java.net/browse/JDK-8213577

Thanks,
Xuelei

Reply via email to