jmx-dev [PATCH] JDK-6809322: Missing notifications from javax.management.timer.Timer

[Dmitry Samersoff]

Jaroslav,

Not able to open it as a webrev - only list of files.

E-mail me the webrev and I'll put it to

cr.openjdk.net/~dsamersoff/sponsorship/jbachorik/NNNNN

-Dmitry


On 2012-10-10 18:39, Jaroslav Bachorik wrote:
> I am sorry for the webrev URL - a stale clipboard :(
> 
> The correct webrev URL is
> https://github.com/jbachorik/openjdk-patches/tree/master/webrevs/JDK-6809322
> 
> -JB-
> 
> On 10/10/2012 04:17 PM, Jaroslav Bachorik wrote:
>> I am looking for a review and a sponsor.
>>
>> The issue is about some javax.management.timer.Timer notifications not
>> being received by the listeners if the notifications are generated rapidly.
>>
>> The problem is caused by ConcurrentModificationException being thrown -
>> the exception itself is ignored but the dispatcher logic is skipped.
>> Therefore the currently processed notification gets lost.
>>
>> The CME is thrown due to the Timer.timerTable being iterated over while
>> other threads try to remove some of its elements. Fix consists of
>> replacing the Hashtable used for Timer.timerTable by ConcurrentHashMap
>> which handles such situations with grace.
>>
>> The patch webrev is available @
>> https://jbs.oracle.com/bugs/browse/JDK-6809322
>>
>> Thanks,
>>
>> -JB-
>>
> 


-- 
Dmitry Samersoff
Java Hotspot development team, SPB04
* There will come soft rains ...