On Wed, 6 Nov 2024 12:03:56 GMT, Prasadrao Koppula <pkopp...@openjdk.org> wrote:
>> Using SharedSecrets, I attempted to expose FileInputStream::path >> information. After implementing the fix, I validated the startup performance >> tests. Observed no consistent pattern of performance drops or gains, can >> disregard the occasional performance drop observed in 1 or 2 runs. > > Prasadrao Koppula has updated the pull request incrementally with one > additional commit since the last revision: > > Replaced Paths.get with Path.of This looks good to me. I'd suggest logging a new JBS issue to track removal of the Debug.getInstance("pkcs12") use case and substituting it with getInstance("keystore") code (esp. since pkcs12 is now the default keystore type) ------------- Marked as reviewed by coffeys (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/20414#pullrequestreview-2418430330