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

[Jaroslav Bachorik]

Thanks, could you update with this webrev? I've fixed the problem with
hgmq <-> webrev where the file copy is mistakenly marked as a file move.

Thanks,

-JB-

On 10/10/2012 08:32 PM, Dmitry Samersoff wrote:
> 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-
>>>>>
>>>>
>>>
>>>
>>
>>
> 
>