Re: RFR: 8331142: Add test for number of loader threads in BasicDirectoryModel [v2]

2024-04-30 Thread Tejesh R
On Mon, 29 Apr 2024 19:59:18 GMT, Alexey Ivanov wrote: >> This PR provides a regression test for >> [JDK-8325179](https://bugs.openjdk.org/browse/JDK-8325179): _Race in >> BasicDirectoryModel.validateFileCache_ reviewed in #18111. >> >> The test is inspired and based on

Re: RFR: 8331142: Add test for number of loader threads in BasicDirectoryModel [v2]

2024-04-29 Thread Alexey Ivanov
> This PR provides a regression test for > [JDK-8325179](https://bugs.openjdk.org/browse/JDK-8325179): _Race in > BasicDirectoryModel.validateFileCache_ reviewed in #18111. > > The test is inspired and based on `ConcurrentModification` that I wrote for >

Re: RFR: 8331142: Add test for number of loader threads in BasicDirectoryModel

2024-04-29 Thread Tejesh R
On Mon, 29 Apr 2024 14:45:43 GMT, Tejesh R wrote: > > Is it necessary to create dummy files here in this test? Can't we just > > create JFileChooser without creating dummy files and proceed with loader > > test? Because I tested without using dummy files and getting exception > > without

Re: RFR: 8331142: Add test for number of loader threads in BasicDirectoryModel

2024-04-29 Thread Tejesh R
On Thu, 25 Apr 2024 16:37:39 GMT, Alexey Ivanov wrote: > This PR provides a regression test for > [JDK-8325179](https://bugs.openjdk.org/browse/JDK-8325179): _Race in > BasicDirectoryModel.validateFileCache_ reviewed in #18111. > > The test is inspired and based on `ConcurrentModification`

Re: RFR: 8331142: Add test for number of loader threads in BasicDirectoryModel

2024-04-29 Thread Tejesh R
On Mon, 29 Apr 2024 13:22:00 GMT, Alexey Ivanov wrote: > > Is it necessary to create dummy files here in this test? Can't we just > > create JFileChooser without creating dummy files and proceed with loader > > test? Because I tested without using dummy files and getting exception > > without

Re: RFR: 8331142: Add test for number of loader threads in BasicDirectoryModel

2024-04-29 Thread Alexey Ivanov
On Mon, 29 Apr 2024 10:42:26 GMT, Tejesh R wrote: > Is it necessary to create dummy files here in this test? Can't we just create > JFileChooser without creating dummy files and proceed with loader test? > Because I tested without using dummy files and getting exception without > JDK-8325179

Re: RFR: 8331142: Add test for number of loader threads in BasicDirectoryModel

2024-04-29 Thread Tejesh R
On Thu, 25 Apr 2024 16:37:39 GMT, Alexey Ivanov wrote: > This PR provides a regression test for > [JDK-8325179](https://bugs.openjdk.org/browse/JDK-8325179): _Race in > BasicDirectoryModel.validateFileCache_ reviewed in #18111. > > The test is inspired and based on `ConcurrentModification`

Re: RFR: 8331142: Add test for number of loader threads in BasicDirectoryModel

2024-04-25 Thread Sergey Bylokhov
On Thu, 25 Apr 2024 16:37:39 GMT, Alexey Ivanov wrote: > This PR provides a regression test for > [JDK-8325179](https://bugs.openjdk.org/browse/JDK-8325179): _Race in > BasicDirectoryModel.validateFileCache_ reviewed in #18111. > > The test is inspired and based on `ConcurrentModification`

Re: RFR: 8331142: Add test for number of loader threads in BasicDirectoryModel

2024-04-25 Thread Alexey Ivanov
On Thu, 25 Apr 2024 16:37:39 GMT, Alexey Ivanov wrote: > This PR provides a regression test for > [JDK-8325179](https://bugs.openjdk.org/browse/JDK-8325179): _Race in > BasicDirectoryModel.validateFileCache_ reviewed in #18111. > > The test is inspired and based on `ConcurrentModification`

RFR: 8331142: Add test for number of loader threads in BasicDirectoryModel

2024-04-25 Thread Alexey Ivanov
This PR provides a regression test for [JDK-8325179](https://bugs.openjdk.org/browse/JDK-8325179): _Race in BasicDirectoryModel.validateFileCache_ reviewed in #18111. The test is inspired and based on `ConcurrentModification` that I wrote for