Integrated: 8354565: jtreg failure handler GatherProcessInfoTimeoutHandler has a leftover call to System.loadLibrary

Jaikiran Pai jpai at openjdk.org
Wed Apr 16 02:00:46 UTC 2025


On Tue, 15 Apr 2025 05:50:03 GMT, Jaikiran Pai <jpai at 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 <jpai at 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


More information about the build-dev mailing list