On Wed, 7 Jul 2021 12:55:32 GMT, Ivan Šipka <isi...@openjdk.org> wrote:
>> @dfuch could you please review, thank you.
>
> Ivan Šipka has updated the pull request with a new target base due to a merge
> or a rebase. The pull request now contains seven commits:
>
> - 8263364: output
> - 8263364: removing surplus imports
> - 8263364: refactor
> - 8263364: refactor
> - 8263364: refactor
> - 8263364: fixed headers
> - JDK-8263364: initial commit
Ah - I see that there are still problems with this test:
#section:main
----------messages:(4/225)----------
command: main KeepAliveStreamCloseWithWrongContentLength
reason: User specified action: run main/othervm/timeout=30
KeepAliveStreamCloseWithWrongContentLength
Mode: othervm [/othervm specified]
elapsed time (seconds): 0.206
----------configuration:(0/0)----------
----------System.out:(9/153)----------
client reads: 49
client reads: 50
client reads: 51
client reads: 52
client reads: 53
client reads: 54
client reads: 55
client reads: 56
client reads: 57
----------System.err:(25/1821)----------
java.net.SocketException: Connection reset
at java.base/sun.nio.ch.NioSocketImpl.implRead(NioSocketImpl.java:322)
at java.base/sun.nio.ch.NioSocketImpl.read(NioSocketImpl.java:347)
at java.base/sun.nio.ch.NioSocketImpl$1.read(NioSocketImpl.java:800)
at java.base/java.net.Socket$SocketInputStream.read(Socket.java:966)
at
java.base/java.io.BufferedInputStream.fill(BufferedInputStream.java:244)
at
java.base/java.io.BufferedInputStream.read1(BufferedInputStream.java:284)
at
java.base/java.io.BufferedInputStream.read(BufferedInputStream.java:343)
at java.base/sun.net.www.MeteredStream.read(MeteredStream.java:141)
at java.base/java.io.FilterInputStream.read(FilterInputStream.java:132)
at
java.base/sun.net.www.protocol.http.HttpURLConnection$HttpInputStream.read(HttpURLConnection.java:3663)
at
java.base/sun.net.www.protocol.http.HttpURLConnection$HttpInputStream.read(HttpURLConnection.java:3656)
at
java.base/sun.net.www.protocol.http.HttpURLConnection$HttpInputStream.read(HttpURLConnection.java:3644)
at
KeepAliveStreamCloseWithWrongContentLength.main(KeepAliveStreamCloseWithWrongContentLength.java:144)
at
java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at
java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
at
java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.base/java.lang.reflect.Method.invoke(Method.java:568)
at
com.sun.javatest.regtest.agent.MainWrapper$MainThread.run(MainWrapper.java:127)
at java.base/java.lang.Thread.run(Thread.java:833)
JavaTest Message: Test threw exception: java.net.SocketException: Connection
reset
JavaTest Message: shutting down test
STATUS:Failed.`main' threw exception: java.net.SocketException: Connection reset
That would need to be fixed before integrating.
-------------
PR: https://git.openjdk.java.net/jdk/pull/4472
