Codereview request: 8050115 javax/management/monitor/GaugeMonitorDeadlockTest.java fails intermittently

David Holmes david.holmes at oracle.com
Wed Sep 17 00:22:18 UTC 2014


On 17/09/2014 7:01 AM, shanliang wrote:
> David Holmes wrote:
>> Hi Shanliang,
>>
>> On 16/09/2014 7:12 PM, shanliang wrote:
>>> Hi,
>>>
>>> Please review the following fix:
>>
>> I don't see any functional change. You seem to have replaced a
>> built-in timeout with the externally applied test harness timeout.
> Yes no functional change here, we thought that the test needed more time
> to wait a change if a testing VM or machine was really slow, the test
> harness timeout was the maximum time we could give the test.

Do we have confidence that the harness timeout is sufficient to handle 
the intermittent failures?

Thanks,
David


>>
>> Style nit: add a space after 'while' -> while (cond) {
> OK, I will do it before pushing.
>
> Thanks,
> Shanliang
>>
>> David
>> -----
>>
>>> bug: https://bugs.openjdk.java.net/browse/JDK-8050115
>>> webrev: http://cr.openjdk.java.net/~sjiang/JDK-8050115/00/
>>>
>>> Thanks,
>>> Shanliang
>


More information about the serviceability-dev mailing list