On Fri, 19 Apr 2024 17:06:54 GMT, robert engels <d...@openjdk.org> wrote:

>> 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.
>
>> 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.
> 
> I don't understand this in reference to the other tests failing because they 
> are not closing the output stream or ending the exchange. If that is 
> incorrect behavior - because they are sending content - then those tests are 
> invalid.
> 
> Still I have a fix for this I believe.

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)

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

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

Reply via email to