On Mon, May 8, 2017 at 1:57 PM, Christian Schneider <ch...@die-schneider.net> wrote: > Hi Zoran, > > I have merged your improvements into the branch. > > I will wait one more day for feedback and then merge into master. >
Can you wait some longer. Some of us have been traveling all last week and are back in our office with jet-lag and a ton of email to catch up. I will take a look at this later this week. > Christian > > > On 08.05.2017 11:57, Zoran Regvart wrote: >> >> Hi Christian, >> I think that your initial example of the recursion is a simpler >> example for the unit test, so with that and a minor reformat of the >> source code I've committed this to my GitHub fork: >> >> >> https://github.com/zregvart/camel/commit/d76f75ea46b55b16af9e0ae05bfbb3df271545bd >> >> have a look, thanks :) >> >> zoran >> >> On Fri, May 5, 2017 at 5:43 PM, Christian Schneider >> <ch...@die-schneider.net> wrote: >>> >>> I have pushed a test and a possible fix to >>> https://github.com/apache/camel/tree/CAMEL-11229 >>> >>> As the code around error handling is pretty difficult to understand I am >>> not >>> sure if my solution is good. I would be happy if someone could review my >>> change. >>> >>> Christian >>> >>> >>> On 04.05.2017 17:30, Christian Schneider wrote: >>>> >>>> I have the routes below. When I send a message to direct:test I get an >>>> infinite recursion of exceptions. >>>> The reason is that the onException handler also seems to be called for >>>> the >>>> direct:handle_er that is called when handling the >>>> first exception. In case such a handler route also throws an exception >>>> the >>>> recursion happens. >>>> >>>> Is this expected or a bug? >>>> If it is not a bug what do we recommend our users to avoid the >>>> recursion? >>>> >>>> onException(Throwable.class) >>>> .to("direct:handle_er"); >>>> >>>> from("direct:test") >>>> .throwException(new RuntimeException()) >>>> .to("log:test2"); >>>> >>>> from("direct:handle_er") >>>> .throwException(new RuntimeException()); >>>> >>>> See also https://issues.apache.org/jira/browse/CAMEL-11229 >>>> >>>> Christian >>>> >>> >>> -- >>> Christian Schneider >>> http://www.liquid-reality.de >>> >>> Open Source Architect >>> http://www.talend.com >>> >> >> > > > -- > Christian Schneider > http://www.liquid-reality.de > > Open Source Architect > http://www.talend.com > -- Claus Ibsen ----------------- http://davsclaus.com @davsclaus Camel in Action 2: https://www.manning.com/ibsen2
