Camel
  1. Camel
  2. CAMEL-3534

Stopping a route has side effects on adviced routes

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.6.0
    • Fix Version/s: 2.7.3, 2.8.0
    • Component/s: camel-core
    • Labels:
      None

      Issue Links

        Activity

        Hide
        Claus Ibsen added a comment -

        CAMEL-3878 fixes this issue as well.

        Show
        Claus Ibsen added a comment - CAMEL-3878 fixes this issue as well.
        Hide
        John Fansler added a comment -

        You are correct Claus, moving the stop to before the advice makes the issue go away. I had set it up this way because I normally do the advice configuration in the "setup" call for a testing class. I then created a number of different tests/methods within the class. This causes the route to be restarted before each test, thus my call to stop it after the advice. Maybe this isn't the best way to setup my normal tests.

        Show
        John Fansler added a comment - You are correct Claus, moving the stop to before the advice makes the issue go away. I had set it up this way because I normally do the advice configuration in the "setup" call for a testing class. I then created a number of different tests/methods within the class. This causes the route to be restarted before each test, thus my call to stop it after the advice. Maybe this isn't the best way to setup my normal tests.
        Hide
        Claus Ibsen added a comment -

        And btw you should stop the route before you do the advice, then it may not cause side effects

        Show
        Claus Ibsen added a comment - And btw you should stop the route before you do the advice, then it may not cause side effects
        Hide
        Claus Ibsen added a comment -

        We will revisit this when we work on Camel 3.0 having a more dynamic engine when applying interceptors, error handlers at runtime

        Show
        Claus Ibsen added a comment - We will revisit this when we work on Camel 3.0 having a more dynamic engine when applying interceptors, error handlers at runtime
        Hide
        John Fansler added a comment -

        Attached is the unit test I have to recreate the issue.

        Show
        John Fansler added a comment - Attached is the unit test I have to recreate the issue.

          People

          • Assignee:
            Claus Ibsen
            Reporter:
            John Fansler
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development