RFR: 8376533: Remove test dependencies on ReferenceQueue$Lock in preparation for JDK-8376477
Eirik Bjørsnøs
eirbjo at openjdk.org
Wed Jan 28 08:41:20 UTC 2026
On Tue, 27 Jan 2026 21:11:05 GMT, Eirik Bjørsnøs <eirbjo at openjdk.org> wrote:
> Please review this PR which updates a few JDK tests to not depend on the existence of `java.lang.ref.ReferenceQueue$Lock` and `java.lang.Shutdown$Lock` classes. These are suggested for removal in JDK-8376477, see #29442.
>
> * `ClhsdbInspect` is updated to use add a nested Lock class in the sample app and use that instead.
> * `ConcurrentAssociateTest` and `Phaser/Basic` are updated to not use lock name when filtering threads to dump for debugging purposes when failing.
> * `ImageReaderBenchmark` micro is updated to remove these classes from the list of expected system image resource names
>
> I have verified that these tests now run with or without the `Lock` classes present in the JDK. I called the thread dump debug methods manually to verify them since these only run on test failure. GHA run passes.
>
> Test only change/cleanup. `noreg-self`
Note that `ConcurrentAssociateTest` and `Phaser/Basic` additionally depends on the existence of the long gone class `java.lang.ref.Reference$Lock`.
This class was removed by JDK-8156500 back in 2016, so these dump methods have been slightly "broken" in their filtering since then. This PR cleans up that as well.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/29455#issuecomment-3809806810
More information about the core-libs-dev
mailing list