RFR: 8253178: Replace LinkedList Impl in net.http.FilterFactory [v3]
Daniel Fuchs
dfuchs at openjdk.java.net
Mon Aug 23 15:20:38 UTC 2021
On Mon, 19 Jul 2021 10:24:27 GMT, Sergei Ustimenko <github.com+5709644+fdesu at 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
More information about the net-dev
mailing list