jmx-dev JDK-7120365 DiffHBTest.java fails due to ConcurrentModificationException

Alan Bateman Alan.Bateman at oracle.com
Wed Dec 26 13:14:57 PST 2012


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