Hi David and Alex,
Thank for reviewing this change.
I will push it to jdk15 repo as David suggested.
Best regards,
Daniil
On 6/11/20, 5:28 PM, "Alex Menkov" <alexey.men...@oracle.com> wrote:
+1
--alex
On 06/11/2020 16:51, David Holmes wrote:
> Hi Daniil,
>
> On 12/06/2020 5:56 am, Daniil Titov wrote:
>> Please review change [1] that fixes an intermittent failure of the
>> test when it is runs with -Xcomp.
>>
>> The problem here is that the timespan the test uses to count
>> notifications is not adjusted for "test.timeout.factor" system property.
>
> The adjustment looks fine.
>
>> The original issue is reproducible in JDK 11 and on Solaris platform
>> only. However, I think it makes sense to apply this change in JDK 15
>> to prevent this from possible happening in the future and then
>> backport it to 11.
>
> Do you still intend this for 15 or just 16? If 15 then push to jdk15
> repo and it will get forward ported to jdk/jdk automatically.
>
> Thanks,
> David
>
>> [1] http://cr.openjdk.java.net/~dtitov/8246196/webrev.01/
>> [2] https://bugs.openjdk.java.net/browse/JDK-8246196
>>
>> Thank you,
>> Daniil
>>
>>
>>
