RFR: 6968351: httpserver clashes with delayed TCP ACKs for low Content-Length [v4]

[Daniel Jeliński]

On Fri, 19 Apr 2024 17:51:55 GMT, robert engels <duke at openjdk.org> wrote:

>> ah no, this is not related to the test failures; this is only about the original issue (small sends result in problems with delayed acks)
>
>> The problem with closing a chunked stream is that it flushes both before and after the final chunk. I assume it may also cause delays.
>> 
>> The only case when you don't need to close the exchange or the output stream is when your response contains no content.
> 
> Can you point me to the code covering the before and after the final chunk? I would think the before flush should be removed.

https://github.com/openjdk/jdk/blob/b704e91241b0f84d866f50a8f2c6af240087cb29/src/jdk.httpserver/share/classes/sun/net/httpserver/ChunkedOutputStream.java#L140-L144

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18667#discussion_r1572745437