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

After switching to slf4j, we can get rid of the 'isTraceEnabled', 'isDebugEnabled' and 'isInfoEnabled' statements

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.6.0
    • 2.8.0
    • None
    • None

    Description

      we can get rid of the 'isTraceEnabled', 'isDebugEnabled' and 'isInfoEnabled' statements with slf4j and use

      logger.debug("Temperature set to {}. Old temperature was {}.", t, oldT);

      instead

      christian-muellers-macbook-pro:camel cmueller$ egrep -r 'isTraceEnabled|isDebugEnabled|isInfoEnabled]' . | wc -l
      1485

      Attachments

        Issue Links

          is related to

          Improvement - An improvement or enhancement to an existing feature or task. CAMEL-2229 switch to slf4j / logback by default as we'll get accurate version information in stack traces, plus MDC support (and neater ways to avoid the "if (LOG.isDebugEnabled())" stuff

          • Major - Major loss of function.
          • Closed
          links to

          Web Link GitHub Pull Request #1134

          Activity

            People

              muellerc Christian Müller
              muellerc Christian Müller
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: