RFR: 8286873: Improve websocket test execution time

Daniel Jeliński djelinski at openjdk.java.net
Tue May 17 12:53:21 UTC 2022


This PR improves the execution time of jdk_net tests (and, by extension, tier2) by about 3 minutes.

Tests located under `jdk/java/net/httpclient/websocket` are never run in parallel. Each of the 8 modified `Pending***` tests originally required 40 seconds to complete. After the proposed changes, they usually complete in 15 seconds.

This PR modifies the tests to initially run with 1 second timeout. If the test fails with 1 second timeout, it is retried with timeout increased to 10 seconds (the original value).

The modified tests were executed at least 10 times on each of: Windows, Linux (both x64 and aarch64), MacOS (both x64 and aarch64). No failures were observed.

-------------

Commit messages:
 - Explain magic timeouts
 - Retry all operations
 - Increase initial wait for Windows machines
 - Fix test retry
 - Avoid one 5-second wait
 - Use shorter timeout on the first try

Changes: https://git.openjdk.java.net/jdk/pull/8746/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=8746&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8286873
  Stats: 139 lines in 10 files changed: 70 ins; 38 del; 31 mod
  Patch: https://git.openjdk.java.net/jdk/pull/8746.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/8746/head:pull/8746

PR: https://git.openjdk.java.net/jdk/pull/8746


More information about the net-dev mailing list