Hi Sameera,

On Mon, Mar 19, 2012 at 1:25 PM, Sameera Jayasoma <[email protected]> wrote:

>
>
> On Sun, Mar 18, 2012 at 1:24 AM, Sameera Jayasoma <[email protected]>wrote:
>
>> Hi Devs,
>>
>> There were number of incidents where users have reported issues related
>> to logging, when they tried to add a new log4j.properties file to their
>> webapps, services, etc. And Carbon initialization code(Non-OSGi) does not
>> use logging, instead System.out were used. It is because of the issues
>> we've been facing with commons-logging & log4j in non-OSGi and OSGi spaces.
>>  With these latest changes to Carbon, we thought of fixing this
>> long-running issue properly.
>>
>> Carbon OSGi container is launched by the org.wso2.carbon.server package
>> which is running in non-OSGi world. This code does not use logging at all,
>> simply uses System.out. We need to to user proper logging there also. Here
>> is the proposal to fix these logging related issues.
>>
>> Current log4j.properties file contains the TenantAwarePatternLayout which
>> is written by us. This class comes from an OSGi bundle. But the problem is
>> we cannot use this property file for the Carbon initialization code. We
>> need to a simple and separate one without the TenantAwarePatternLayout.
>>
>> CARBON_HOME
>> |--- lib
>>      |---- log4j.properties (For Carbon initialization code.)
>>      |---- org.wso2.carbon.server-4.0.0-SHAPSHOT.jar
>>
>
>  After a offline discussion with Azeez, we decided to embed the
> log4j.property file required for the carbon initialization code.
>

+1. Much cleaner approach. Having two log4j.properties files would've
easily confused someone.

Thanks,
Senaka.

>
> Thanks,
> Sameera.
>
>>
>> |--- repository
>>      |---- conf
>>           |---- log4j.properties
>>
>> IMHO, log4j.properties file of the Carbon platform should be placed
>> inside the repository/conf. Its the correct place to put it as other config
>> files.
>>
>> With these changes, I could successfully deploy a webapp with a separate
>> log4j.properties file.
>>
>> Thanks,
>> Sameera.
>> --
>> Sameera Jayasoma
>> Technical Lead and Product Manager, WSO2 Carbon
>>
>> WSO2, Inc. (http://wso2.com)
>> email: [email protected]
>> blog: http://tech.jayasoma.org
>>
>> Lean . Enterprise . Middleware
>>
>
>
>
> --
> Sameera Jayasoma
> Technical Lead and Product Manager, WSO2 Carbon
>
> WSO2, Inc. (http://wso2.com)
> email: [email protected]
> blog: http://tech.jayasoma.org
>
> Lean . Enterprise . Middleware
>
> _______________________________________________
> Dev mailing list
> [email protected]
> http://wso2.org/cgi-bin/mailman/listinfo/dev
>
>


-- 
*Senaka Fernando*
Product Manager - WSO2 Governance Registry;
Associate Technical Lead; WSO2 Inc.; http://wso2.com*
Member; Apache Software Foundation; http://apache.org

E-mail: senaka AT wso2.com
**P: +1 408 754 7388; ext: 51736*; *M: +94 77 322 1818
Linked-In: http://linkedin.com/in/senakafernando

*Lean . Enterprise . Middleware
_______________________________________________
Dev mailing list
[email protected]
http://wso2.org/cgi-bin/mailman/listinfo/dev

Reply via email to