Re: [11] Review Request: 8198342 Test FileSystemViewListenerLeak.java is unstable

2018-04-25 Thread Jayathirth D V
Hi Sergey, Thanks for the clarification. Changes are fine. Regards, Jay -Original Message- From: Sergey Bylokhov Sent: Thursday, April 26, 2018 5:48 AM To: Jayathirth D V; swing-dev@openjdk.java.net Subject: Re: [11] Review Request: 8198342 Test FileSystemViewListenerLeak.java is

Re: [11] Review Request: 8198342 Test FileSystemViewListenerLeak.java is unstable

2018-04-25 Thread Sergey Bylokhov
Hi, Jay. On 24/04/2018 23:27, Jayathirth D V wrote: I went through the changes done in JDK-8175968 and I can understand why you would have added creation of multiple CustomFileSystemView like stress test scenario. But as you have mentioned in highly loaded systems there can sometimes be proble

Re: [11] Review Request: 8198342 Test FileSystemViewListenerLeak.java is unstable

2018-04-24 Thread Jayathirth D V
case. Thanks, Jay -Original Message- From: Sergey Bylokhov Sent: Wednesday, April 25, 2018 3:43 AM To: swing-dev@openjdk.java.net Subject: Re: [11] Review Request: 8198342 Test FileSystemViewListenerLeak.java is unstable Any volunteers to review? =) On 12/03/2018 22:27, Sergey Bylokhov

Re: [11] Review Request: 8198342 Test FileSystemViewListenerLeak.java is unstable

2018-04-24 Thread Prasanta Sadhukhan
+1 Regards Prasanta On 4/25/2018 3:43 AM, Sergey Bylokhov wrote: Any volunteers to review? =) On 12/03/2018 22:27, Sergey Bylokhov wrote: Hello. Please review the update of the test for jdk11. Bug: https://bugs.openjdk.java.net/browse/JDK-8198342 Webrev can be found at: http://cr.openjdk.jav

Re: [11] Review Request: 8198342 Test FileSystemViewListenerLeak.java is unstable

2018-04-24 Thread Krishna Addepalli
Looks fine to me. -Original Message- From: Sergey Bylokhov Sent: Wednesday, April 25, 2018 3:43 AM To: swing-dev@openjdk.java.net Subject: Re: [11] Review Request: 8198342 Test FileSystemViewListenerLeak.java is unstable Any volunteers to review? =) On 12/03/2018 22:27, Sergey

Re: [11] Review Request: 8198342 Test FileSystemViewListenerLeak.java is unstable

2018-04-24 Thread Sergey Bylokhov
Any volunteers to review? =) On 12/03/2018 22:27, Sergey Bylokhov wrote: Hello. Please review the update of the test for jdk11. Bug: https://bugs.openjdk.java.net/browse/JDK-8198342 Webrev can be found at: http://cr.openjdk.java.net/~serb/8198342/webrev.00 This test creates a number of FileSys

[11] Review Request: 8198342 Test FileSystemViewListenerLeak.java is unstable

2018-03-12 Thread Sergey Bylokhov
Hello. Please review the update of the test for jdk11. Bug: https://bugs.openjdk.java.net/browse/JDK-8198342 Webrev can be found at: http://cr.openjdk.java.net/~serb/8198342/webrev.00 This test creates a number of FileSystemView which internally register a listeners in UIManager, and the test a