[jdk17u-dev] RFR: 8254711: Add java.security.Provider.getService JFR Event [v2]

[Goetz Lindenmaier]

On Thu, 20 Apr 2023 09:15:39 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:

>> Goetz Lindenmaier has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Backport 3644e26cef71c00e1a2638d2b8bed9c1bda965ca
>
> Hi Götz, regarding Provider.java, do we need the  legacyMap.remove  that can be found in head ?  I do not see it in your coding, but maybe it is not needed in 17 ?

Hi @MBaesken, 
In head, legacyMap.remove is not added by the change. It is only embraced
in another if (s == null).  It was added by "8276660: Scalability bottleneck in java.security.Provider.getService()"
which is a rather large change not suitable for 17 I think.

-------------

PR Comment: https://git.openjdk.org/jdk17u-dev/pull/1260#issuecomment-1519591417