Re: RFR JDK-8158980: Memory leak in HTTP2Connection.streams

2016-06-23 Thread Chris Hegarty
Pushed: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/bae5b550b829 -Chris. > On 22 Jun 2016, at 22:17, Sergey Kuksenko wrote: > > Please, do this. > > On 06/20/2016 10:17 PM, Chris Hegarty wrote: >>> On 20 Jun 2016, at 21:24, Sergey Kuksenko >>> wrote: >>> >>> Hi, >>> >>> Could you please

Re: RFR JDK-8158980: Memory leak in HTTP2Connection.streams

2016-06-22 Thread Sergey Kuksenko
Please, do this. On 06/20/2016 10:17 PM, Chris Hegarty wrote: On 20 Jun 2016, at 21:24, Sergey Kuksenko wrote: Hi, Could you please review the following fix for JDK-8158980? http://cr.openjdk.java.net/~skuksenko/jep110/8158980/webrev.00/ This looks good to me Sergey. I can sponsor it for y

Re: RFR JDK-8158980: Memory leak in HTTP2Connection.streams

2016-06-20 Thread Chris Hegarty
> On 20 Jun 2016, at 21:24, Sergey Kuksenko wrote: > > Hi, > > Could you please review the following fix for JDK-8158980? > http://cr.openjdk.java.net/~skuksenko/jep110/8158980/webrev.00/ This looks good to me Sergey. I can sponsor it for you. -Chris.

RFR JDK-8158980: Memory leak in HTTP2Connection.streams

2016-06-20 Thread Sergey Kuksenko
Hi, Could you please review the following fix for JDK-8158980? http://cr.openjdk.java.net/~skuksenko/jep110/8158980/webrev.00/ Fix solves the following issue: https://bugs.openjdk.java.net/browse/JDK-8158980 Stream will be never deleted from HTTP2Connection.streams in the following condition

RFR JDK-8158980: Memory leak in HTTP2Connection.streams

2016-06-20 Thread Sergey Kuksenko
Hi, Could you please review the following fix for JDK-8158980? http://cr.openjdk.java.net/~skuksenko/jep110/8158980/webrev.00/ Fix solves the following issue: https://bugs.openjdk.java.net/browse/JDK-8158980 Stream will be never deleted from HTTP2Connection.streams in the following condition