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. This pull request has now been integrated. Changeset: 55264907 Author: Jaikiran Pai <j...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/5526490743a75786a40754b6805f4381ae2892b5 Stats: 12 lines in 1 file changed: 0 ins; 12 del; 0 mod 8354565: jtreg failure handler GatherProcessInfoTimeoutHandler has a leftover call to System.loadLibrary Reviewed-by: alanb, kbarrett ------------- PR: https://git.openjdk.org/jdk/pull/24648
