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

Claus Ibsen commented on CAMEL-9153:
------------------------------------

Yeah that can be a good idea. Wonder if you can add an unit test if possible? 
Also the coding style we use is to have } else { on the same line.

There is a page here how to build with checkstyle that checks for the code style
http://camel.apache.org/building.html

> 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, 
> ThreadPoolRejectedPolicy-AbortNonRejectable.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)

Reply via email to