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