Re: RFR: 8088343: Race condition in javafx.concurrent.Task::cancel [v3]

2025-04-24 Thread Andy Goryachev
On Wed, 16 Apr 2025 20:28:14 GMT, Andy Goryachev wrote: >> The code should not set the `Task.state` value to `CANCELLED` if the said >> task is already `SUCCEEDED` or `FAILED`. >> >> This is a product bug. >> >> Added `@RepeatedTest(50)` to the tests that used to fail intermittently - >> this

Re: RFR: 8088343: Race condition in javafx.concurrent.Task::cancel [v3]

2025-04-24 Thread Ambarish Rapte
On Wed, 16 Apr 2025 20:28:14 GMT, Andy Goryachev wrote: >> The code should not set the `Task.state` value to `CANCELLED` if the said >> task is already `SUCCEEDED` or `FAILED`. >> >> This is a product bug. >> >> Added `@RepeatedTest(50)` to the tests that used to fail intermittently - >> this

Re: RFR: 8088343: Race condition in javafx.concurrent.Task::cancel [v3]

2025-04-23 Thread Andy Goryachev
On Wed, 16 Apr 2025 20:28:14 GMT, Andy Goryachev wrote: >> The code should not set the `Task.state` value to `CANCELLED` if the said >> task is already `SUCCEEDED` or `FAILED`. >> >> This is a product bug. >> >> Added `@RepeatedTest(50)` to the tests that used to fail intermittently - >> this

Re: RFR: 8088343: Race condition in javafx.concurrent.Task::cancel [v3]

2025-04-23 Thread Kevin Rushforth
On Wed, 16 Apr 2025 20:28:14 GMT, Andy Goryachev wrote: >> The code should not set the `Task.state` value to `CANCELLED` if the said >> task is already `SUCCEEDED` or `FAILED`. >> >> This is a product bug. >> >> Added `@RepeatedTest(50)` to the tests that used to fail intermittently - >> this