On Tue, 15 Apr 2025 05:50:03 GMT, Jaikiran Pai <j...@openjdk.org> wrote:
> Can I please get a review of this cleanup in the jtreg timeout handler > `GatherProcessInfoTimeoutHandler`? > > As noted in https://bugs.openjdk.org/browse/JDK-8354565, this is a leftover > after the change done in https://bugs.openjdk.java.net/browse/JDK-8268626. > The `timeoutHandler` native test library is no longer built in the JDK, nor > it is required by this `GatherProcessInfoTimeoutHandler`. > > I've verified that the `GatherProcessInfoTimeoutHandler` continues to > function normally on macosx (x64/aarch64), linux(x64/aarch64) and windows > (x64), by intentionally timing out a test and verifying that the > `GatherProcessInfoTimeoutHandler` continues to gather the relevant > diagnostics. Looks good. ------------- Marked as reviewed by kbarrett (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/24648#pullrequestreview-2766895709
