That looks fine to me now.

Thanks,
Michael

On 24/02/2017, 10:28, Pavel Rappo wrote:
Michael, Daniel,

updated in place.

Thanks,
-Pavel

On 23 Feb 2017, at 17:31, Michael McMahon<michael.x.mcma...@oracle.com>  wrote:

Hi Pavel

The source change looks fine. I'm not sure how the test works though.
How does it check that the connection is not retained?

Michael.

On 23/02/2017, 14:02, Pavel Rappo wrote:
Hello,

Could you please review the following change for [1]?

   http://cr.openjdk.java.net/~prappo/8164625/webrev.00/

This patch addresses the root cause of the problem. Namely, HTTP connections
used by requests made on behalf of the WebSocket (i.e. Opening Handshake) are no
longer returned to the connection pool. The provided test reproduces the
scenario described in the related issue.

Thanks,
-Pavel

--------------------------------------------------------------------------------
[1] https://bugs.openjdk.java.net/browse/JDK-8164625

Reply via email to