You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Velmurugan Periasamy <vp...@hortonworks.com> on 2016/04/15 02:44:30 UTC

Re: Review Request 45027: RANGER-887 : Changes needed to convert log4j.xml to log4j.properties

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45027/#review129050
-----------------------------------------------------------


Ship it!




Ship It!

- Velmurugan Periasamy


On March 31, 2016, 7 a.m., Mehul Parikh wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45027/
> -----------------------------------------------------------
> 
> (Updated March 31, 2016, 7 a.m.)
> 
> 
> Review request for ranger, Alok Lal, Don Bosco Durai, Gautam Borad, Abhay Kulkarni, Madhan Neethiraj, Ramesh Mani, and Selvamohan Neethiraj.
> 
> 
> Bugs: RANGER-887
>     https://issues.apache.org/jira/browse/RANGER-887
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> **Problem Statement**
> In order to allow log4j to be exposed to Ambari and configurable from Ambari, we need to convert log4j.xml file to log4j.properties file.
> 
> **Proposed Solution**
> Making required changes in pom.xml (usersync) and web.xml(ranger-admin) to use log4j.properties and converting log4j.xml to log4j.properties.
> 
> 
> Diffs
> -----
> 
>   security-admin/src/main/webapp/WEB-INF/log4j.properties PRE-CREATION 
>   security-admin/src/main/webapp/WEB-INF/web.xml 0725633 
>   tagsync/conf.dist/log4j.properties PRE-CREATION 
>   tagsync/scripts/ranger-tagsync-services.sh 058ebf3 
>   tagsync/scripts/setup.py 6d28cd2 
>   unixauthservice/conf.dist/log4j.properties PRE-CREATION 
>   unixauthservice/pom.xml edff076 
>   unixauthservice/scripts/ranger-usersync-services.sh 622c986 
> 
> Diff: https://reviews.apache.org/r/45027/diff/
> 
> 
> Testing
> -------
> 
> ** Testing Done**
> 
> Verified manual install as well as ambari based installs of Ranger to use log4j.properties to log required activities.
> 
> 
> Thanks,
> 
> Mehul Parikh
> 
>