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