Re: RFR: 8129988: JSSE should create a single instance of the cacerts KeyStore

2015-09-30 Thread Xuelei Fan
On 9/30/2015 9:44 PM, Sean Mullan wrote: > On 9/29/15 10:33 PM, Xuelei Fan wrote: >> private synchronized static KeyStore getDefaultCacertsKeyStore() >> >> >> --- >> private synchronized static KeyStore getDefaultCacertsKe

Re: RFR: 8129988: JSSE should create a single instance of the cacerts KeyStore

2015-09-30 Thread Sean Mullan
On 9/29/15 10:33 PM, Xuelei Fan wrote: private synchronized static KeyStore getDefaultCacertsKeyStore() --- private synchronized static KeyStore getDefaultCacertsKeyStore( String javaHome, S

Re: RFR: 8129988: JSSE should create a single instance of the cacerts KeyStore

2015-09-29 Thread Xuelei Fan
private synchronized static KeyStore getDefaultCacertsKeyStore() --- private synchronized static KeyStore getDefaultCacertsKeyStore( String javaHome, String type, St

RFR: 8129988: JSSE should create a single instance of the cacerts KeyStore

2015-09-29 Thread Sean Mullan
Please review this fix to modify the TrustManagerFactory implementation to create a single instance of the cacerts or jssecacerts KeyStore. This significantly improves performance in a multithreaded environment. The code has been refactored a bit to move common code into a few private methods.