jmx-dev JDK-7120365 DiffHBTest.java fails due to ConcurrentModificationException
shanliang
shanliang.jiang at oracle.com
Thu Dec 27 03:33:51 PST 2012
Thanks for all comments, here is the new webrev:
http://cr.openjdk.java.net/~sjiang/JDK-7120365/webrev.03/
Indeed, no need to have @run for the test.
Shanliang
Alan Bateman wrote:
> On 26/12/2012 15:07, shanliang wrote:
>> Yes should use a cop[y, it is a mistake to use a unmodifiable view.
>>
>> Here is the new webrev:
>> http://cr.openjdk.java.net/~sjiang/JDK-7120365/webrev.02/
>>
>> I have added a new test to reproduce the bug in an almost sure way.
>>
>> Thanks,
>>
>> Shanliang
> Thanks for the update, the synchronization and copy looks right to me.
> An alternative you be "if (delegateSet == null ||
> delegateSet.isEmpty()) return; " which might be a bit cleaner.
>
> I didn't study the new test very closely but I assume you should only
> catch MalformedURLException at line 122 rather than Exception. As it's
> a new test then I assume it should have 2012 as the date. You can
> probably remove the @run as the defaults (compile + run) is right for
> this test.
>
> -Alan
More information about the serviceability-dev
mailing list