RFR: 8293000: Review running times of jshell regression tests

Adam Sotona asotona at openjdk.org
Mon Sep 26 17:28:46 UTC 2022


Some of the jshell tests have been identified as long-running.

This patch reduces tests runtimes with multiple tricks:

- local execution control is used in heavily parametrized and fragmented tests to eliminate JVM startups
- some tests have been defragmented
- unnecessary Thread::stop has been removed

Measured test run times before and after application of this patch:
- ClassMembersTest 23.964ms ->  9.195ms
- ToolBasicTest 24.281ms -> 14.955ms
- FailOverExecutionControlHangingListenTest 31.221ms -> 5.780ms
- FailOverExecutionControlDyingLaunchTest 31.221ms -> 6.050ms
- FailOverExecutionControlHangingLaunchTest 31.303ms -> 5.832ms
- CompletionSuggestionTest 10.523ms -> 4.940ms
- SimpleRegressionTest 3.629ms -> 1.304ms

Please review the test changes.

Thanks,
Adam

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

Commit messages:
 - 8293000: Review running times of jshell regression tests

Changes: https://git.openjdk.org/jdk/pull/10428/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=10428&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8293000
  Stats: 42 lines in 5 files changed: 23 ins; 10 del; 9 mod
  Patch: https://git.openjdk.org/jdk/pull/10428.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/10428/head:pull/10428

PR: https://git.openjdk.org/jdk/pull/10428


More information about the kulla-dev mailing list