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

[Jaikiran Pai]

On Tue, 23 Apr 2024 19:10:48 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:
> 
>   fix broken test cases

test/jdk/com/sun/net/httpserver/bugs/TcpNoDelayNotRequired.java line 32:

> 30:  */
> 31: 
> 32: import com.sun.net.httpserver.*;

It would be better to use exact imports instead of wildcard.

test/jdk/com/sun/net/httpserver/bugs/TcpNoDelayNotRequired.java line 44:

> 42: import java.util.concurrent.ExecutorService;
> 43: import java.util.concurrent.Executors;
> 44: import java.util.logging.*;

Same here, exact imports instead of wildcard.

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

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