[jira] [Created] (CAMEL-12004) Add LOG.isDebugEnabled() guard for LOG.debug()

["Mehran Hassani (JIRA)" <ji...@apache.org>]

Mehran Hassani created CAMEL-12004:
--------------------------------------

             Summary: Add LOG.isDebugEnabled() guard for LOG.debug()
                 Key: CAMEL-12004
                 URL: https://issues.apache.org/jira/browse/CAMEL-12004
             Project: Camel
          Issue Type: Improvement
            Reporter: Mehran Hassani
            Priority: Minor


I am conducting research on log related bugs. I tried to make a tool to fix repetitive yet simple patterns of bugs that are related to logs. In these files, there are debug level logging statements containing multiple string concatenation or method calls without the if statement before them:


camel-core/src/test/java/org/apache/camel/LanguageTestSupport.java, log.debug("Evaluated expression: " + expression + " on exchange: " + exchange + " result: " + value);, 97
components/camel-github/src/test/java/org/apache/camel/component/github/consumer/CommitConsumerTest.java, log.debug("Got commit with author: " + author.getLogin() + ": " + author.getHtmlUrl() + " SHA " + commit.getSha());, 62
components/camel-jms/src/main/java/org/apache/camel/component/jms/reply/QueueReplyManager.java, log.debug("Using shared queue: " + endpoint.getReplyTo() + " with fixed message selector [" + fixedMessageSelector + "] as reply listener: " + answer);, 133
components/camel-spring/src/main/java/org/apache/camel/spring/spi/TransactionErrorHandler.java, log.debug("Transaction rollback (" + transactionKey + ") redelivered(" + redelivered + ") for "+ ids + " due exchange was marked for rollbackOnlyLast and caught: ", cause);, 157
components/camel-test/src/main/java/org/apache/camel/test/junit4/LanguageTestSupport.java, log.debug("Evaluated expression: " + expression + " on exchange: " + exchange + " result: " + value);, 88
components/camel-testng/src/main/java/org/apache/camel/testng/LanguageTestSupport.java, log.debug("Evaluated expression: " + expression + " on exchange: " + exchange + " result: " + value);, 88
tooling/maven/camel-maven-plugin/src/main/java/org/apache/camel/maven/RunMojo.java, getLog().debug("strange; " + activeCount + " thread(s) still active in the group "+ threadGroup + " such as " + threadsArray[0]);, 652
tooling/maven/camel-maven-plugin/src/main/java/org/apache/camel/maven/RunMojo.java, getLog().debug("Skipping org.osgi.core -> " + classPathElement.getGroupId() + "/" + classPathElement.getArtifactId() + "/" + classPathElement.getVersion());, 761
tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/PackageDataFormatMojo.java, log.debug("Generated " + out + " containing JSon schema for " + name + " data format");, 218
tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/PackageHelper.java, log.debug("checking  if " + path + " (" + r.getDirectory() + "/" + suffix + ") has changed.");, 48
tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/PackageLanguageMojo.java, log.debug("Generated " + out + " containing JSon schema for " + name + " language");, 215
tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/SpringBootStarterMojo.java, getLog().debug("Found dependency node: " + artifact.getGroupId() + ":" + artifact.getArtifactId() + ":" + artifact.getVersion() + " - scope=" + artifact.getScope());, 401
tooling/maven/guice-maven-plugin/src/main/java/org/apache/camel/guice/maven/RunMojo.java, getLog().debug("strange; " + activeCount + " thread(s) still active in the group "+ threadGroup + " such as " + threadsArray[0]);, 515

Would you be interested in adding the if before these logging statements?






--
This message was sent by Atlassian JIRA
(v6.4.14#64029)