camel-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Christian Müller (JIRA) <j...@apache.org>
Subject [jira] [Updated] (CAMEL-6202) Missing log parameter placeholder in routepolicy.quartz.ScheduledRoutePolicy
Date Fri, 22 Mar 2013 22:07:15 GMT

     [ https://issues.apache.org/jira/browse/CAMEL-6202?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Christian Müller updated CAMEL-6202:
------------------------------------

      Description: 
In the method scheduleRoute(...) there is logging code that looks like:

{code}
  if (LOG.isInfoEnabled()) {
    LOG.info("Scheduled trigger: {} for action: {} on route: ", 
        new Object[]{trigger.getFullName(), action, route.getId()});
  }
{code}

...but it's missing parameter placeholder for the routeId.  It should be:

{code}
  if (LOG.isInfoEnabled()) {
    LOG.info("Scheduled trigger: {} for action: {} on route: {}", 
        new Object[]{trigger.getFullName(), action, route.getId()});
  }
{code}


  was:

In the method scheduleRoute(...) there is logging code that looks like:

{code}
  if (LOG.isInfoEnabled()) {
    LOG.info("Scheduled trigger: {} for action: {} on route: ", 
        new Object[]{trigger.getFullName(), action, route.getId()});
  }
{code}

...but it's missing parameter placeholder for the routeId.  It should be:

{code}
  if (LOG.isInfoEnabled()) {
    LOG.info("Scheduled trigger: {} for action: {} on route: {}", 
        new Object[]{trigger.getFullName(), action, route.getId()});
  }
{code}


    Fix Version/s: 2.11.0
                   2.10.5
                   2.9.7
    
> Missing log parameter placeholder in routepolicy.quartz.ScheduledRoutePolicy
> ----------------------------------------------------------------------------
>
>                 Key: CAMEL-6202
>                 URL: https://issues.apache.org/jira/browse/CAMEL-6202
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-quartz
>    Affects Versions: 2.10.4
>            Reporter: Chris Wolf
>            Assignee: Christian Müller
>            Priority: Trivial
>              Labels: logging
>             Fix For: 2.9.7, 2.10.5, 2.11.0
>
>
> In the method scheduleRoute(...) there is logging code that looks like:
> {code}
>   if (LOG.isInfoEnabled()) {
>     LOG.info("Scheduled trigger: {} for action: {} on route: ", 
>         new Object[]{trigger.getFullName(), action, route.getId()});
>   }
> {code}
> ...but it's missing parameter placeholder for the routeId.  It should be:
> {code}
>   if (LOG.isInfoEnabled()) {
>     LOG.info("Scheduled trigger: {} for action: {} on route: {}", 
>         new Object[]{trigger.getFullName(), action, route.getId()});
>   }
> {code}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message