On Mon, 19 Jul 2021 10:24:27 GMT, Sergei Ustimenko 
<github.com+5709644+fd...@openjdk.org> wrote:

>> This patch replaces a LinkedList data structure used in the 
>> net.http.FilterFactory class with an ArrayList. This issue relates to 
>> [JDK-8246048: Replace LinkedList with ArrayLists in 
>> java.net.](https://bugs.openjdk.java.net/browse/JDK-8246048).
>> 
>> The list created once per HttpClient and filled with upfront known values (3 
>> of them in the jdk.internal.net.http.HttpClientImpl#initFilters: 
>> AuthenticationFilter.class, RedirectFilter.class and depending on the 
>> presence of a cookieHandler - a CookieFilter.class).
>
> Sergei Ustimenko has updated the pull request with a new target base due to a 
> merge or a rebase. The incremental webrev excludes the unrelated changes 
> brought in by the merge/rebase. The pull request contains two additional 
> commits since the last revision:
> 
>  - 8253178: Trim the size for list with headers
>  - 8253178: Replace LinkedList Impl in net.http.FilterFactory

LGTM. I can sponsor once you have integrated
.

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

Marked as reviewed by dfuchs (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/4721

Reply via email to