Re: Codereview requested: 6566891 RMIConnector: map value referencing map key in WeakHashMap prevents map entry to be removed

2013-08-29 Thread shanliang
Thanks David. Shanliang David Holmes wrote: Hi Shanliang, After our off-list discussion I've had a closer look at this and it looks okay to me too. Reviewed. Thanks, David On 26/08/2013 6:35 PM, shanliang wrote: Hi, Still need OK from a reviewer. Thanks Erik, Jaroslav and Daniel for

Re: Codereview requested: 6566891 RMIConnector: map value referencing map key in WeakHashMap prevents map entry to be removed

2013-08-26 Thread shanliang
Hi, Still need OK from a reviewer. Thanks Erik, Jaroslav and Daniel for the code review. Thanks, Shanliang Daniel Fuchs wrote: On 8/21/13 10:27 AM, shanliang wrote: Jaroslav Bachorik wrote: On 08/21/2013 12:00 AM, Daniel Fuchs wrote: On 8/20/13 11:12 PM, shanliang wrote: Thanks Daniel

Re: Codereview requested: 6566891 RMIConnector: map value referencing map key in WeakHashMap prevents map entry to be removed

2013-08-21 Thread shanliang
Jaroslav Bachorik wrote: On 08/21/2013 12:00 AM, Daniel Fuchs wrote: On 8/20/13 11:12 PM, shanliang wrote: Thanks Daniel and Erik for the comments, here is new version: http://cr.openjdk.java.net/~sjiang/JDK-6566891/01/ This new version looks reasonable. I guess you could

Re: Codereview requested: 6566891 RMIConnector: map value referencing map key in WeakHashMap prevents map entry to be removed

2013-08-21 Thread Jaroslav Bachorik
On 08/21/2013 10:27 AM, shanliang wrote: Jaroslav Bachorik wrote: On 08/21/2013 12:00 AM, Daniel Fuchs wrote: On 8/20/13 11:12 PM, shanliang wrote: Thanks Daniel and Erik for the comments, here is new version: http://cr.openjdk.java.net/~sjiang/JDK-6566891/01/ This new

Codereview requested: 6566891 RMIConnector: map value referencing map key in WeakHashMap prevents map entry to be removed

2013-08-20 Thread shanliang
Hi, Please review: webrev: http://amos.fr.oracle.com/jmgt/user/sjiang/webrevs/jdk8-6566891/00/ bug: https://jbs.oracle.com/bugs/browse/JDK-6566891 I have passed JCK tests and unit tests. Thanks, Shanliang

Re: Codereview requested: 6566891 RMIConnector: map value referencing map key in WeakHashMap prevents map entry to be removed

2013-08-20 Thread shanliang
webrev: http://cr.openjdk.java.net/~sjiang/JDK-6566891/00/ shanliang wrote: Hi, Please review: webrev: http://amos.fr.oracle.com/jmgt/user/sjiang/webrevs/jdk8-6566891/00/ bug: https://jbs.oracle.com/bugs/browse/JDK-6566891 I have passed JCK tests and unit tests. Thanks, Shanliang

Re: Codereview requested: 6566891 RMIConnector: map value referencing map key in WeakHashMap prevents map entry to be removed

2013-08-20 Thread Erik Gahlin
Shanliang, Let's say the delgationSubject is null and there is previous connection cached (nullSubjectConn != null nullSubjectConn.get() != null) so you hit: 2016: conn = nullSubjectConn.get(); but before you get the object a GC occurs and clears reference, then getConnectionWithSubject

Re: Codereview requested: 6566891 RMIConnector: map value referencing map key in WeakHashMap prevents map entry to be removed

2013-08-20 Thread shanliang
Thanks Daniel and Erik for the comments, here is new version: http://cr.openjdk.java.net/~sjiang/JDK-6566891/01/ Erik Gahlin wrote: Shanliang, Let's say the delgationSubject is null and there is previous connection cached (nullSubjectConn != null nullSubjectConn.get() != null) so you