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

[Daniel Jeliński]

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

>> fix bug JDK-B6968351 by avoiding flush after response headers
>
> robert engels has updated the pull request incrementally with one additional commit since the last revision:
> 
>   update based on PR review

test/jdk/com/sun/net/httpserver/bugs/B6968351.java line 30:

> 28:  * @library /test/lib
> 29:  * @run main/timeout=5 B6968351
> 30:  * @run main/othervm -Dsun.net.httpserver.nodelay=false B6968351

Suggestion:

 * @run main/othervm/timeout=5 -Dsun.net.httpserver.nodelay=false B6968351

you need the timeout on every `run` line.

The test will be run as many times as there are `run` lines; I'm assuming you actually want to run it twice here, correct?

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

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