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

[Dmitry Samersoff]

Jaroslav,

http://cr.openjdk.java.net/~dsamersoff/sponsorship/jbachorik/JDK-6809322/

-Dmitry


On 2012-10-10 22:25, Jaroslav Bachorik wrote:
> On Wed 10 Oct 2012 05:42:43 PM CEST, Dmitry Samersoff wrote:
>> Jaroslav,
>>
>> Not able to open it as a webrev - only list of files.
>>
>> E-mail me the webrev and I'll put it to
>> file:///opt/src/jdks/openjdk-patches/webrevs/JDK-6809322.zip
> 
>> cr.openjdk.net/~dsamersoff/sponsorship/jbachorik/NNNNN
> 
> Attaching... Thanks. I didn't want to put the webrevs on rapidshare or 
> the likes and github seemed like a nice choice. Unfortunately it is not 
> possible to view raw files :(
> 
> -JB-
> 
>>
>> -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 ...