[prev in list] [next in list] [prev in thread] [next in thread] 

List:       activemq-dev
Subject:    [jira] Assigned: (AMQ-1235) Scheduler.cancel uses incorrect
From:       "james strachan (JIRA)" <jira () apache ! org>
Date:       2007-04-30 12:56:34
Message-ID: 8573767.1177937794761.JavaMail.jira () brutus
[Download RAW message or body]


     [ https://issues.apache.org/activemq/browse/AMQ-1235?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel \
]

james strachan reassigned AMQ-1235:
-----------------------------------

    Assignee: Hiram Chirino

> Scheduler.cancel uses incorrect argument to shutdown threads
> ------------------------------------------------------------
> 
> Key: AMQ-1235
> URL: https://issues.apache.org/activemq/browse/AMQ-1235
> Project: ActiveMQ
> Issue Type: Bug
> Affects Versions: 4.1.1
> Reporter: Alex Burgel
> Assigned To: Hiram Chirino
> Priority: Critical
> Attachments: scheduler.patch
> 
> 
> looking at the code from 4.1.1 in org.apache.activemq.thread.Scheduler, in the \
> cancel method: the Runnable task argument is passed to clockDaemon.remove(). i \
> think this is incorrect. ScheduledFuture ticket should be passed to \
> clockDaemon.remove(). the javadocs of ScheduledThreadPoolExecutor.remove discuss \
> the possibility that Runnables might be stored in some other form internally, so \
> calling remove with a plain Runnable might not do anything. I think the solution is \
> to call remove with a ScheduledFuture, which is how they are stored internally in \
> ScheduledThreadPoolExecutor. i came across this bug after upgrading to the java 5 \
> version of backport-util-concurrent 3.0. that version makes more assumptions about \
> the types that are passed into ScheduledThreadPoolExecutor.remove, so when you pass \
> in a regular Runnable you'll get a ClassCastException. this is trivial to fix, so i \
> don't think a patch is necessary. also i think this might address the memory leak \
> mentioned in AMQ-1205

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[prev in list] [next in list] [prev in thread] [next in thread]