[ 
https://issues.apache.org/jira/browse/GERONIMO-6911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18126589#comment-18126589
 ] 

Thomas Butz commented on GERONIMO-6911:
---------------------------------------

Related: https://github.com/apache/activemq/issues/2647

> GeronimoWorkManager doesn't handle RejectedExecutionException
> -------------------------------------------------------------
>
>                 Key: GERONIMO-6911
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-6911
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>            Reporter: Thomas Butz
>            Priority: Major
>
> According to the Javadoc for the WorkManager interface, all methods throw 
> either a WorkException or a WorkRejectedException, which is a subclass of 
> WorkException. However, the current implementation of 
> GeronimoWorkManager.executeWork() does not handle a 
> RejectedExecutionException from its executors, causing it to be escalated to 
> the invoker.
>  
> In my opinion, this is a leaking implementation detail and should be covered 
> by an additional catch clause alongside the InterruptedException. It should 
> be wrapped in a WorkRejectedException.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to