[jdk8u-dev] RFR: 8293562: KeepAliveCache Blocks Threads while Closing Connections [v3]

[Severin Gehwolf]

On Thu, 8 Feb 2024 22:28:29 GMT, Dhamoder Nalla <dhanalla at openjdk.org> wrote:

>> Issue: [JDK-8293562](https://bugs.openjdk.org/browse/JDK-8293562): KeepAliveCache Blocks Threads while Closing Connections
>> Backport of PR https://github.com/openjdk/jdk11u-dev/pull/1890 fromJDK11 to JDK8.
>> 
>> The JDK11 patch doesn't apply cleanly 3/11 hunks need to be ported manually in KeepAliveCache.java.
>> 
>> Test performed:
>> 
>> 1. JTreg jdk_tier1
>> 2. JTreg jdk_tier2
>> 3. New test case added to this PR passed with the fix and failed without it.
>> 4. The customer failure scenario has been validated.
>
> Dhamoder Nalla has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains three commits:
> 
>  - Merge pr/437
>  - Fix indentation
>  - Backport 770c1f65c588f3156f9b70097df752d8059c1038

If we are going to backport this, then I think we should use the following approach in order to keep the risk manageable (and to keep in sync with what had been done in JDK 11):

1. Get `JDK-8278067: Make HttpURLConnection default keep alive timeout configurable` into JDK 8u. That's #437 
2. Get `JDK-8291226: Create Test Cases to cover scenarios for JDK-8278067` backported as well as it includes correctness tests for `JDK-8293562` as well.
3. Get `JDK-8291638: Keep-Alive timeout of 0 should close connection immediately` backported.

-------------

PR Comment: https://git.openjdk.org/jdk8u-dev/pull/409#issuecomment-1956384254