RFR: 8348880: Replace ConcurrentMap with AtomicReferenceArray for ZoneOffset.MINUTES_15_CACHE [v5]
Naoto Sato
naoto at openjdk.org
Tue Jan 28 23:57:45 UTC 2025
On Tue, 28 Jan 2025 17:06:24 GMT, Shaojin Wen <swen at openjdk.org> wrote:
>> ZoneOffset.MINUTES_15_CACHE uses AtomicReferenceArray to replace ConcurrentMap to avoid object allocation caused by boxing from int to Integer during access.
>
> Shaojin Wen has updated the pull request incrementally with one additional commit since the last revision:
>
> from @liach
The change looks good to me. Maybe you could write a test case for all the quarters that indeed return cached ZoneOffsets? Also, titles (PR/Issue) can reflect the renamed cache name
-------------
PR Comment: https://git.openjdk.org/jdk/pull/23337#issuecomment-2620289635
More information about the core-libs-dev
mailing list