camel-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Riedel (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CAMEL-9153) ThreadPoolRejectedPolicy does not implement Abort as expected
Date Wed, 23 Sep 2015 13:48:08 GMT

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

Michael Riedel commented on CAMEL-9153:
---------------------------------------

I attached a patch now, which would make the {{RejectedExecutionHandler}} of {{ThreadPoolRejectedPolicy.Abort}}
throw a {{RejectedExecutionException}}. After notifying the {{Rejectable}} that is. 

Not sure if that is an appropriate solution. I have to admit that I have not tested it. However,
it seems more in line with older versions of the implementation, in particular, versions before
_4e745ad_.

> ThreadPoolRejectedPolicy does not implement Abort as expected
> -------------------------------------------------------------
>
>                 Key: CAMEL-9153
>                 URL: https://issues.apache.org/jira/browse/CAMEL-9153
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-core
>            Reporter: Michael Riedel
>         Attachments: ThreadPoolRejectedPolicy-Abort.patch
>
>
> Not sure if this is a bug or just a documentation flaw (in fact I could not find documentation
for it). Nevertheless, as a naive user, I kind of expected that the {{RejectedExecutionHandler}}
of {{ThreadPoolRejectedPolicy.Abort}} would have similar semantics as Java's {{ThreadPoolExecutor.AbortPolicy}}.
That is, I expected it to throw an {{Exception}}.
> Currently (that is in the 2.15.2 release, and on the master branch in git) that is not
the case. In fact, {{Abort}} seems to work exactly the same way as {{Discard}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message