[13] RFR JDK-7107615 "scalability bloker in javax.crypto.JceSecurity"

[Alan Bateman]

On 15/05/2019 20:11, Valerie Peng wrote:
> I updated the webrev to switch to ConcurrentHashMap. The javadoc spec 
> of computeIfAbsent method cautioned that the mapping func should be 
> short and simple and must not attempt to update other mappings of this 
> map. Provider verification code does not quite fit the above criteria 
> for the mapping. So, I did not use computeIfAbsent method and just 
> made minor update to webrev.01 with Xuelei's suggestion of re-checking 
> the cache again inside the synchronized block.
>
> http://cr.openjdk.java.net/~valeriep/7107615/webrev.03/
>
> Comments?
Does it need to synchronize on JceSecurity.class? I'm mostly wondering 
if it can use computeIfAbsent.

-Alan