Re: RFR: Remove map synchronization from SignatureAndHashAlgorithm

2017-04-26 Thread Sean Mullan
On 4/26/17 1:52 PM, Steven Davidovitz wrote: Thanks! I'm working on the OCA right now, I made a mistake when I submitted it before so I'll follow up when it's done. Anything else I need to do? Could this also be backported to version 8? We are already at RDP2 [1] for JDK 9. Only P1 and P2 bugs

Re: RFR: Remove map synchronization from SignatureAndHashAlgorithm

2017-04-26 Thread Steven Davidovitz
Thanks! I'm working on the OCA right now, I made a mistake when I submitted it before so I'll follow up when it's done. Anything else I need to do? Could this also be backported to version 8? On Tue, Apr 25, 2017 at 7:05 AM, Sean Mullan wrote: > Hi Steven, > > Thanks, I filed an issue on your b

Re: RFR: Remove map synchronization from SignatureAndHashAlgorithm

2017-04-25 Thread Sean Mullan
Hi Steven, Thanks, I filed an issue on your behalf for tracking purposes: https://bugs.openjdk.java.net/browse/JDK-8179275 Also, have you signed the OCA? See http://www.oracle.com/technetwork/community/oca-486395.html for more information. --Sean On 4/15/17 3:25 PM, Steven Davidovitz wrot

RFR: Remove map synchronization from SignatureAndHashAlgorithm

2017-04-15 Thread Steven Davidovitz
With the removal of the synchronization on priorityMap inside 'SignatureAndHashAlgorithm.getSupportedAlgorithms' in rev daaace32c979 [1], it seems unnecessary to use a synchronizedSortedMap. Benchmarking, I see a 2x performance increase by using the bare TreeMap. measureModified sample 11336330