On Tue, 27 Sep 2022 19:00:28 GMT, Daniel Jeliński <[email protected]> wrote:
>> Please review this patch that makes sure KeepAliveCache does not block all >> threads while closing sockets. >> >> Changes: >> - get operation no longer closes sockets; if there's no socket that is >> recent enough, get returns null and lets the cleaner thread close the sockets >> - put operation closes sockets without holding the cache lock. Additionally, >> if the cache is full, it places the new connection in the cache and removes >> the oldest connection. >> - the cleaner thread creates a list of connections to close, and then closes >> them after releasing the cache lock >> - additionally, we set the socket timeout to 1 millisecond before calling >> socket.close >> >> The new test fails with `Wait for second request timed out` without this >> patch, passes after the changes. Tiers 1-3 clean. > > Daniel Jeliński has updated the pull request incrementally with one > additional commit since the last revision: > > Stop using getLocalhost test/jdk/sun/net/www/http/KeepAliveCache/B8293562.java line 2: > 1: /* > 2: * Copyright (c) 2005, 2022, Oracle and/or its affiliates. All rights > reserved. Hello Daniel, since this is a new file, the copyright year would just be `2022`. ------------- PR: https://git.openjdk.org/jdk/pull/10401
