On Wed, Feb 2, 2011 at 9:28 PM, Tharindu Mathew <[email protected]> wrote:

> These sort of logs should be at the debug level. Please pay attention to
> the level of your log:
>
> if (log.isInfoEnabled()) {
>                 log.info("BAM Core bundle is activated");
>

+1.

Moreover
if (log.isInfoEnabled()) check above is pretty redundant and overdoing,
which should better be followed only for the log.isDebugEnabled(), I feel.


Thank you.
Regards,
Pradeeban.

 }
>
> --
> Regards,
>
> Tharindu
>
> _______________________________________________
> Carbon-dev mailing list
> [email protected]
> http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev
>
>


-- 
Kathiravelu Pradeeban.
Software Engineer.
WSO2 Inc.

Blog: [Llovizna] http://kkpradeeban.blogspot.com/
_______________________________________________
Carbon-dev mailing list
[email protected]
http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev

Reply via email to