Re: RFR: 8263364: sun/net/www/http/KeepAliveStream/KeepAliveStreamCloseWithWrongContentLength.java wedged in getInputStream [v7]
On Mon, 6 Sep 2021 18:18:21 GMT, Ivan Šipka 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 incremental webrev excludes the unrelated changes brought in > by the merge/rebase. The pull request contains 10 additional commits since > the last revision: > > - whitespaces removed > - Merge branch 'master' into JDK-8263364-keep-alive-should-not-throw-null > - drain channel impl > - 8263364: output > - 8263364: removing surplus imports > - 8263364: refactor > - 8263364: refactor > - 8263364: refactor > - 8263364: fixed headers > - JDK-8263364: initial commit Thanks for fixing. If you /integrate drop me a note and I will sponsor. - PR: https://git.openjdk.java.net/jdk/pull/4472
Re: RFR: 8263364: sun/net/www/http/KeepAliveStream/KeepAliveStreamCloseWithWrongContentLength.java wedged in getInputStream [v7]
On Tue, 7 Sep 2021 09:25:31 GMT, Daniel Fuchs wrote: >> Ivan Šipka has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains 10 additional >> commits since the last revision: >> >> - whitespaces removed >> - Merge branch 'master' into JDK-8263364-keep-alive-should-not-throw-null >> - drain channel impl >> - 8263364: output >> - 8263364: removing surplus imports >> - 8263364: refactor >> - 8263364: refactor >> - 8263364: refactor >> - 8263364: fixed headers >> - JDK-8263364: initial commit > > Thanks for fixing. If you /integrate drop me a note and I will sponsor. thank you @dfuch - PR: https://git.openjdk.java.net/jdk/pull/4472
Integrated: 8263364: sun/net/www/http/KeepAliveStream/KeepAliveStreamCloseWithWrongContentLength.java wedged in getInputStream
On Fri, 11 Jun 2021 10:46:32 GMT, Ivan Šipka wrote: > @dfuch could you please review, thank you. This pull request has now been integrated. Changeset: 5caa77b0 Author:Ivan Šipka Committer: Daniel Fuchs URL: https://git.openjdk.java.net/jdk/commit/5caa77b043ae490c3d7d56d181d0e07e6b859b9e Stats: 125 lines in 1 file changed: 84 ins; 13 del; 28 mod 8263364: sun/net/www/http/KeepAliveStream/KeepAliveStreamCloseWithWrongContentLength.java wedged in getInputStream Reviewed-by: dfuchs - PR: https://git.openjdk.java.net/jdk/pull/4472
