Uploaded image for project: 'Camel'
  1. Camel
  2. CAMEL-6264

Camel 2.10 performance regression with added statistics in BaseTypeConverterRegistry

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.10.3, 2.11.0
    • 2.10.5, 2.11.0
    • camel-core
    • Unknown

    Description

      We updated one of our applications from camel 2.7.3 to camel 2.10.3.
      We have a camel route that takes its input from a servlet and we saw a 10% performance drop after the update.

      With more tests using yourkit, it appears that we are spending a lot more time in BaseTypeConverterRegistry#convertTo method than we used to (from 10ms to 3200ms in a test case).

      The only difference between the two version is the added statistics and the AtomicLong that must cause contention on our route that uses multiple threads behind a seda.

      To improve the issue on our side, we worked-around to avoid having a type converter in our hotspots (Which was mainly the DefaultHttpBinding#writeResponse from camel-http), but i think a solution should be find (maybe make these statistics optionnal).

      Regards,
      Fabien

      Attachments

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            davsclaus Claus Ibsen
            fchaillou Fabien Chaillou
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment