RFR 8068591: javax/management/remote/mandatory/notif/NotifReconnectDeadlockTest.java should be quarantined

Jaroslav Bachorik jaroslav.bachorik at oracle.com
Thu Jan 8 16:43:47 UTC 2015


On 8.1.2015 17:37, shanliang wrote:
> Jaroslav Bachorik wrote:
>> On 8.1.2015 15:35, shanliang wrote:
>>> Jaroslav Bachorik wrote:
>>>> Hi Shanliang,
>>>>
>>>> On 8.1.2015 14:59, shanliang wrote:
>>>>> Hi,
>>>>>
>>>>> Please review this simple fix:
>>>>>
>>>>> bug: https://bugs.openjdk.java.net/browse/JDK-8068591
>>>>> webrev: http://cr.openjdk.java.net/~sjiang/JDK-8068591/00/
>>>>
>>>> The entry is missing the label (eg. "generic-all")
>>> Oops!
>>>     http://cr.openjdk.java.net/~sjiang/JDK-8068591/01/
>>
>> Hm, shouldn't
>> javax/management/remote/mandatory/connection/ReconnectTest.java be
>> quarantined too?
> Maybe, but better to have a new sub-bug of JDK-8042215 to quarantine
> ReconnectTest.java?

Well, it's up to you. But the error report is originally for 
ReconnectTest nd it already mixes those two failures together anyway so 
you could spare one round of one-liner review process.

If you decide to to squeeze in the 
javax/management/remote/mandatory/connection/ReconnectTest.java 
exclusion just reply to this thread with the updated webrev. Otherwise 
this change is good to go.


-JB-

>
> Shanliang
>>
>> -JB-
>>
>>>
>>> Thanks,
>>> Shanliang
>>>>
>>>> -JB-
>>>>
>>>>>
>>>>> Thanks,
>>>>> Shanliang
>>>>
>>>
>>
>



More information about the serviceability-dev mailing list