RFR: 8351996: Behavioral updates for ClassValue::remove [v2]

Viktor Klang vklang at openjdk.org
Mon Apr 28 16:06:47 UTC 2025


On Mon, 28 Apr 2025 04:29:40 GMT, Chen Liang <liach at openjdk.org> wrote:

>> Hello Chen, not a review of the code, but the tier1 failures in the GitHub actions jobs look related:
>> 
>> 
>> java.lang.StackOverflowError: Recursive initialization of class value
>> 	at java.base/java.lang.ClassValue$Entry.registerExtraThread(ClassValue.java:321)
>> 	at java.base/java.lang.ClassValue$ClassValueMap.startEntry(ClassValue.java:481)
>> 	at java.base/java.lang.ClassValue.getFromHashMap(ClassValue.java:196)
>> 	at java.base/java.lang.ClassValue.getFromBackup(ClassValue.java:183)
>> 	at java.base/java.lang.ClassValue.get(ClassValue.java:119)
>
> @jaikiran Since you have looked at this patch and used CountDownLatch frequently before, I wonder if you would like to review the use of concurrency utilities in this patch.

@liach I'll have a look as well.

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

PR Comment: https://git.openjdk.org/jdk/pull/24043#issuecomment-2835753236


More information about the core-libs-dev mailing list