Looks fine to me. BTW, the fix for JDK-8146387 is asking for code review.
http://mail.openjdk.java.net/pipermail/security-dev/2016-January/013301.html Thanks, Xuelei On 1/11/2016 4:54 AM, joe darcy wrote: > Hello, > > The test > > javax/net/ssl/SSLSession/SessionCacheSizeTests.java > > has been failing intermittently with high-frequency on Solaris and > Windows of late. The test should be problem listed until the underlying > problem is fixed (JDK-8146387). > > Patch to do this below. > > Thanks, > > -Joe > > diff -r aa9fd2797b82 test/ProblemList.txt > --- a/test/ProblemList.txt Sun Jan 10 11:09:31 2016 -0800 > +++ b/test/ProblemList.txt Sun Jan 10 12:53:16 2016 -0800 > @@ -1,6 +1,6 @@ > ########################################################################### > > # > -# Copyright (c) 2009, 2015, Oracle and/or its affiliates. All rights > reserved. > +# Copyright (c) 2009, 2016, Oracle and/or its affiliates. All rights > reserved. > # DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. > # > # This code is free software; you can redistribute it and/or modify it > @@ -299,6 +299,9 @@ > # 8074580 > sun/security/pkcs11/rsa/TestKeyPairGenerator.java generic-all > > +# 8146387 > +javax/net/ssl/SSLSession/SessionCacheSizeTests.java > windows-all,solaris-all > + > ############################################################################ > > > # jdk_sound > diff -r aa9fd2797b82 > test/javax/net/ssl/SSLSession/SessionCacheSizeTests.java > --- a/test/javax/net/ssl/SSLSession/SessionCacheSizeTests.java Sun Jan > 10 11:09:31 2016 -0800 > +++ b/test/javax/net/ssl/SSLSession/SessionCacheSizeTests.java Sun Jan > 10 12:53:16 2016 -0800 > @@ -31,6 +31,7 @@ > * @bug 4366807 > * @summary Need new APIs to get/set session timeout and session cache > size. > * @run main/othervm SessionCacheSizeTests > + * @key intermittent > */ > > import java.io.*; > >