On Fri, 11 Aug 2023 13:24:08 GMT, Daniel Fuchs <[email protected]> wrote:
> Please find here a patch for a very rare intermittent failure observed in > CancelRequestTest. > > The fact that the number of pending requests hasn't been decremented leads me > to think that completable future returned by sendAsync (called from send) > hasn't been fully completed. That is, the dependent actions registered by > sendAsync have not been run within the timeout waiting for the number of > pending requests to reach 0. > > The fix is to call cf.get() again after calling cf.cancel(), and increase the > timeout waiting for cleanup in testPostInterrupt. > client.close() is also called at the end of each test method to reclaim > resources earlier This pull request has now been integrated. Changeset: 4f864faf Author: Daniel Fuchs <[email protected]> URL: https://git.openjdk.org/jdk/commit/4f864faf428c8171be975a79db5bc2bc145f8805 Stats: 37 lines in 1 file changed: 7 ins; 8 del; 22 mod 8314136: Test java/net/httpclient/CancelRequestTest.java failed: WARNING: tracker for HttpClientImpl(42) has outstanding operations Reviewed-by: jpai ------------- PR: https://git.openjdk.org/jdk/pull/15249
