RFR: 8255463: java/nio/channels/spi/SelectorProvider/inheritedChannel/InheritedChannelTest.java failed with ThreadTimeoutException
Jaikiran Pai
jpai at openjdk.org
Tue Dec 9 14:44:31 UTC 2025
Can I please get a review of this test-only change which removes the testng specific timeout from a test method?
As noted in https://bugs.openjdk.org/browse/JDK-8255463, the `test/jdk/java/nio/channels/spi/SelectorProvider/inheritedChannel/InheritedChannelTest.java` fails intermittently with:
test InheritedChannelTest.test("StateTest", java.util.ImmutableCollections$List12 at d28f210): failure [30001ms]
org.testng.internal.thread.ThreadTimeoutException: Method InheritedChannelTest.test() didn't finish within the time-out 30000
With the proposed change in this PR, it now relies on jtreg test timeout (default of 120 seconds). If the test continues to timeout for whatever reason, then jtreg will be able to capture the relevant thread dumps of the incomplete process (launched in this test method) to help debug the issue.
The existing 30 second timeout on the test method doesn't play any role in what the test verifies, so removing it doesn't change that verification.
-------------
Commit messages:
- 8255463: java/nio/channels/spi/SelectorProvider/inheritedChannel/InheritedChannelTest.java failed with ThreadTimeoutException
Changes: https://git.openjdk.org/jdk/pull/28721/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=28721&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8255463
Stats: 2 lines in 1 file changed: 0 ins; 1 del; 1 mod
Patch: https://git.openjdk.org/jdk/pull/28721.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/28721/head:pull/28721
PR: https://git.openjdk.org/jdk/pull/28721
More information about the nio-dev
mailing list