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