[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-net-dev
Subject:    Integrated: 8330814: Cleanups for KeepAliveCache tests
From:       Christoph Langer <clanger () openjdk ! org>
Date:       2024-05-15 10:39:12
Message-ID: ofaB6Q8YaLYLt8LwJDMDEeZm4aq0Bte9IMPrPTBCECs=.596961bf-a415-46da-8e38-f212ee68c7ef () github ! com
[Download RAW message or body]

On Mon, 22 Apr 2024 11:06:57 GMT, Christoph Langer <clanger@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@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
[prev in list] [next in list] [prev in thread] [next in thread]