RFR: 8348880: Replace ConcurrentMap with AtomicReferenceArray for ZoneOffset.QUARTER_CACHE [v5]

Shaojin Wen swen at openjdk.org
Wed Jan 29 02:46:57 UTC 2025


On Tue, 28 Jan 2025 23:53:01 GMT, Naoto Sato <naoto at openjdk.org> wrote:

>> 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

Thanks @naotoj, I have changed the title and added a test case to ensure that the cache works.

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

PR Comment: https://git.openjdk.org/jdk/pull/23337#issuecomment-2620507748


More information about the core-libs-dev mailing list