You are viewing a plain text version of this content. The canonical link for it is here.
Posted to users@camel.apache.org by Jason Dillon <ja...@planet57.com> on 2011/12/01 00:28:31 UTC
Odd info logs about startup w/ autoStartup=false
Any reason why these are at INFO level in DefaultCamelContext?
<snip>
if (doNotStartRoutesOnFirstStart) {
log.info("Cannot start routes as CamelContext has been configured with autoStartup=false");
}
</snip>
<snip>
if (addingRoute && !autoStartup) {
log.info("Cannot start route " + routeService.getId() + " as its configured with autoStartup=false");
continue;
}
</snip>
The language sounds like something strange/bad happened, but configuring autoStartup=false isn't strange or bad. I don't get why this detail is logged at INFO.
Its not really a big deal, I was just surprised to see it in my logs after I added a route w/ this set so that I could thread the start of the route in the background.
--jason
Re: Odd info logs about startup w/ autoStartup=false
Posted by Jason Dillon <ja...@planet57.com>.
On Dec 1, 2011, at 1:33 AM, Claus Ibsen wrote:
> Yeah the wording could be improved. Any recommendation?
Is it important to log this information as INFO level? I don't see any of the other attributes of a route logged in this fashion. Is it common that folks set autoStartup(false) and then forget to start manually and wonder why a route is not started?
Wording wise, I think something like this may be better:
"Route %s not started due to configured with autoStartup=false"
or:
"Skipping start of route %s; configured with autoStartup=false"
--jason
Re: Odd info logs about startup w/ autoStartup=false
Posted by Claus Ibsen <cl...@gmail.com>.
Hi
I created a ticket to track this
https://issues.apache.org/jira/browse/CAMEL-4744
On Thu, Dec 1, 2011 at 10:33 AM, Claus Ibsen <cl...@gmail.com> wrote:
> Hi
>
> Yeah the wording could be improved. Any recommendation?
>
>
> On Thu, Dec 1, 2011 at 12:28 AM, Jason Dillon <ja...@planet57.com> wrote:
>> Any reason why these are at INFO level in DefaultCamelContext?
>>
>> <snip>
>> if (doNotStartRoutesOnFirstStart) {
>> log.info("Cannot start routes as CamelContext has been configured with autoStartup=false");
>> }
>> </snip>
>>
>> <snip>
>> if (addingRoute && !autoStartup) {
>> log.info("Cannot start route " + routeService.getId() + " as its configured with autoStartup=false");
>> continue;
>> }
>> </snip>
>>
>> The language sounds like something strange/bad happened, but configuring autoStartup=false isn't strange or bad. I don't get why this detail is logged at INFO.
>>
>> Its not really a big deal, I was just surprised to see it in my logs after I added a route w/ this set so that I could thread the start of the route in the background.
>>
>> --jason
>
>
>
> --
> Claus Ibsen
> -----------------
> FuseSource
> Email: cibsen@fusesource.com
> Web: http://fusesource.com
> Twitter: davsclaus, fusenews
> Blog: http://davsclaus.blogspot.com/
> Author of Camel in Action: http://www.manning.com/ibsen/
--
Claus Ibsen
-----------------
FuseSource
Email: cibsen@fusesource.com
Web: http://fusesource.com
Twitter: davsclaus, fusenews
Blog: http://davsclaus.blogspot.com/
Author of Camel in Action: http://www.manning.com/ibsen/
Re: Odd info logs about startup w/ autoStartup=false
Posted by Claus Ibsen <cl...@gmail.com>.
Hi
Yeah the wording could be improved. Any recommendation?
On Thu, Dec 1, 2011 at 12:28 AM, Jason Dillon <ja...@planet57.com> wrote:
> Any reason why these are at INFO level in DefaultCamelContext?
>
> <snip>
> if (doNotStartRoutesOnFirstStart) {
> log.info("Cannot start routes as CamelContext has been configured with autoStartup=false");
> }
> </snip>
>
> <snip>
> if (addingRoute && !autoStartup) {
> log.info("Cannot start route " + routeService.getId() + " as its configured with autoStartup=false");
> continue;
> }
> </snip>
>
> The language sounds like something strange/bad happened, but configuring autoStartup=false isn't strange or bad. I don't get why this detail is logged at INFO.
>
> Its not really a big deal, I was just surprised to see it in my logs after I added a route w/ this set so that I could thread the start of the route in the background.
>
> --jason
--
Claus Ibsen
-----------------
FuseSource
Email: cibsen@fusesource.com
Web: http://fusesource.com
Twitter: davsclaus, fusenews
Blog: http://davsclaus.blogspot.com/
Author of Camel in Action: http://www.manning.com/ibsen/