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

altrisi duke at openjdk.org
Wed Jan 29 10:40:53 UTC 2025


On Wed, 29 Jan 2025 06:42:02 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:
> 
>   rename to test_quarter_cache

Why not have the key be quarters + 72 so the array can be smaller?

That'd lower the constant memory overhead added by this change, as the CHM was smaller initially.

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

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


More information about the core-libs-dev mailing list