Integrated: 8330814: Cleanups for KeepAliveCache tests

Christoph Langer clanger at openjdk.org
Wed May 15 10:39:12 UTC 2024


On Mon, 22 Apr 2024 11:06:57 GMT, Christoph Langer <clanger at openjdk.org> wrote:

> While working in that area I found some potential for cleanup of a few tests.
> 
> Most notably:
> 
> B5045306.java:
> - does not need to run in othervm
> - the executor service that it uses should be shut down eventually to free resources
> 
> B8291637.java:
> - use just one instead of two test VM invocations
> KeepAliveTimerThread.java:
> call to grp.destroy() at the end is pointless (API is void & deprecated for removal)
> 
> Generally:
> The deprecated URL constructor is used. It can be switched to the handy URIBuilder
> Some more try with resources here and there

This pull request has now been integrated.

Changeset: a5005c87
Author:    Christoph Langer <clanger at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/a5005c87c4d5598eb54e9824105767d833f9660b
Stats:     290 lines in 5 files changed: 113 ins; 69 del; 108 mod

8330814: Cleanups for KeepAliveCache tests

Reviewed-by: jpai, dfuchs

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

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


More information about the net-dev mailing list