RFR: JDK-8310913 Move ReferencedKeyMap to jdk.internal so it may be shared [v6]

Mandy Chung mchung at openjdk.org
Fri Jul 7 16:19:59 UTC 2023


On Fri, 7 Jul 2023 11:49:22 GMT, Jim Laskey <jlaskey at openjdk.org> wrote:

>> java.lang.runtime.ReferencedKeyMap was introduced to provide a concurrent caching scheme for Carrier objects. The technique used is generally useful for a variety of caching schemes and is being moved to be shared in other parts of the jdk. The MethodType interning case is one example.
>
> Jim Laskey has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Simple versions of create

Looks good.  Thanks for the update.   Nit:  the test can use the 2-arg factory method.

test/jdk/jdk/internal/util/ReferencedKeyTest.java line 63:

> 61: 
> 62:     static void mapTest(boolean isSoft, Supplier<Map<ReferenceKey<Long>, String>> supplier) {
> 63:         Map<Long, String> map = ReferencedKeyMap.create(isSoft, false, supplier);

Suggestion:

        Map<Long, String> map = ReferencedKeyMap.create(isSoft, supplier);

test/jdk/jdk/internal/util/ReferencedKeyTest.java line 72:

> 70: 
> 71:     static void setTest(boolean isSoft, Supplier<Map<ReferenceKey<Long>, ReferenceKey<Long>>> supplier) {
> 72:         ReferencedKeySet<Long> set = ReferencedKeySet.create(isSoft, false, supplier);

Suggestion:

        ReferencedKeySet<Long> set = ReferencedKeySet.create(isSoft, supplier);

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

Marked as reviewed by mchung (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/14684#pullrequestreview-1519246419
PR Review Comment: https://git.openjdk.org/jdk/pull/14684#discussion_r1256081914
PR Review Comment: https://git.openjdk.org/jdk/pull/14684#discussion_r1256082313


More information about the core-libs-dev mailing list