On Fri, 19 Apr 2024 15:17:17 GMT, robert engels <d...@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