Integrated: 8269993: [Test]: java/net/httpclient/DigestEchoClientSSL.java contains redundant @run tags

2021-07-07 Thread Thejasvi Voniadka
On Wed, 7 Jul 2021 09:34:13 GMT, Thejasvi Voniadka wrote: > Hi, > > Please review this simple change to remove redundancy of @run tags. The test > "java/net/httpclient/DigestEchoClientSSL.java" contains a couple of redundant > @run tags: > > * @run main/othervm/timeout=300 > * DigestEchoCl

Re: RFR: 8269993: [Test]: java/net/httpclient/DigestEchoClientSSL.java contains redundant @run tags

2021-07-07 Thread Thejasvi Voniadka
On Wed, 7 Jul 2021 14:37:22 GMT, Vyom Tewari wrote: >> Hi, >> >> Please review this simple change to remove redundancy of @run tags. The test >> "java/net/httpclient/DigestEchoClientSSL.java" contains a couple of >> redundant @run tags: >> >> * @run main/othervm/timeout=300 >> * DigestEchoC

Integrated: 8269481: SctpMultiChannel never releases own file descriptor

2021-07-07 Thread Brian Burkhalter
On Tue, 29 Jun 2021 00:44:25 GMT, Brian Burkhalter wrote: > Please review this change to the Unix implementations of > `sun.nio.ch.sctp.Sctp*ChannelImpl#kill()` to close the socket if `state == > ChannelState.UNINITIALIZED`. This pull request has now been integrated. Changeset: d1cecaaa Autho

Re: RFR: 8269993: [Test]: java/net/httpclient/DigestEchoClientSSL.java contains redundant @run tags

2021-07-07 Thread Vyom Tewari
On Wed, 7 Jul 2021 09:34:13 GMT, Thejasvi Voniadka wrote: > Hi, > > Please review this simple change to remove redundancy of @run tags. The test > "java/net/httpclient/DigestEchoClientSSL.java" contains a couple of redundant > @run tags: > > * @run main/othervm/timeout=300 > * DigestEchoCl

Re: RFR: 8263364: sun/net/www/http/KeepAliveStream/KeepAliveStreamCloseWithWrongContentLength.java wedged in getInputStream [v5]

2021-07-07 Thread Ivan Šipka
> @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 seven additional commits since the last revisi

Re: RFR: 8269993: [Test]: java/net/httpclient/DigestEchoClientSSL.java contains redundant @run tags

2021-07-07 Thread Daniel Fuchs
On Wed, 7 Jul 2021 09:34:13 GMT, Thejasvi Voniadka wrote: > Hi, > > Please review this simple change to remove redundancy of @run tags. The test > "java/net/httpclient/DigestEchoClientSSL.java" contains a couple of redundant > @run tags: > > * @run main/othervm/timeout=300 > * DigestEchoCl

Re: RFR: 8269481: SctpMultiChannel never releases own file descriptor [v6]

2021-07-07 Thread Chris Hegarty
On Tue, 6 Jul 2021 16:56:17 GMT, Brian Burkhalter wrote: >> Please review this change to the Unix implementations of >> `sun.nio.ch.sctp.Sctp*ChannelImpl#kill()` to close the socket if `state == >> ChannelState.UNINITIALIZED`. > > Brian Burkhalter has updated the pull request incrementally with

RFR: 8269993: [Test]: java/net/httpclient/DigestEchoClientSSL.java contains redundant @run tags

2021-07-07 Thread Thejasvi Voniadka
Hi, Please review this simple change to remove redundancy of @run tags. The test "java/net/httpclient/DigestEchoClientSSL.java" contains a couple of redundant @run tags: * @run main/othervm/timeout=300 * DigestEchoClientSSL SSL * @run main/othervm/timeout=300 * DigestEchoClientSSL SSL * @r

[jdk17] Integrated: 8269772: [macos-aarch64] test compilation failed with "SocketException: No buffer space available"

2021-07-07 Thread Daniel Fuchs
On Mon, 5 Jul 2021 11:21:39 GMT, Daniel Fuchs wrote: > Please find here a trivial fix for: > > 8269772: [macos-aarch64] test compilation failed with "SocketException: No > buffer space available" > > Running several of the websocket tests concurrently on some of the CI > machines is causing r