On Wed, 17 Aug 2022 17:06:12 GMT, Daniel Fuchs <[email protected]> wrote:
> Please find here a change that improves SpecialHeadersTest. This test creates > a large amount of ephemeral clients and has been observed running out of heap > space in our CI once. This change updates the test to wait for the previous > HttpClient to be eligible for garbage collection before it creates a new one. > It also verifies that no outstanding operation are still running on the > client by the time the client is released. Overall, I understand these changes but I'm still a bit surprised with the way this test failed on the CI system especially after how we try to stop/close the relevant resources (like the SelectorManager thread) as soon as we realize the client is no longer used and doesn't have any pending operations. ------------- PR: https://git.openjdk.org/jdk/pull/9908
