You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Gautam Borad <gb...@gmail.com> on 2016/05/06 16:59:08 UTC

Review Request 47065: RANGER-975 : Rename ranger log file from xa_portal.log to ranger_admin.log

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

Review request for ranger, Don Bosco Durai, Abhay Kulkarni, Madhan Neethiraj, Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, and Velmurugan Periasamy.


Bugs: RANGER-975
    https://issues.apache.org/jira/browse/RANGER-975


Repository: ranger


Description
-------

**Problem Statement:**
xa_portal.log is a very old name and needs to be renamed to ranger_admin for all log files. 


**Proposed Solution: **
1) In ranger_admin dir> ews/logs/  the logs xa_portal.logs has been renamed to ranger_admin.log. 
2) Same as ranger_admin.log the changes are done for ranger_admin_sql_db_patch.log and ranger_admin_sql.log.
3) On plugin side pop-up has been changed with text: ".....Check ranger_admin.log for more info".


Diffs
-----

  agents-common/src/main/java/org/apache/ranger/plugin/client/BaseClient.java 86a91a4 
  hbase-agent/src/main/java/org/apache/ranger/services/hbase/client/HBaseClient.java 1979905 
  hdfs-agent/src/main/java/org/apache/ranger/services/hdfs/client/HdfsClient.java 4d563a3 
  hive-agent/src/main/java/org/apache/ranger/services/hive/client/HiveClient.java e38c26c 
  knox-agent/src/main/java/org/apache/ranger/services/knox/client/KnoxClient.java 8af77b2 
  plugin-kms/src/main/java/org/apache/ranger/services/kms/client/KMSClient.java 271392b 
  plugin-yarn/src/main/java/org/apache/ranger/services/yarn/client/YarnClient.java 0c7bcad 
  security-admin/src/main/java/org/apache/ranger/common/TimedExecutor.java ca2e2dc 
  security-admin/src/main/webapp/WEB-INF/db_patch.log4j.xml 1aafb07 
  security-admin/src/main/webapp/WEB-INF/log4j.dev.xml d1c8b79 
  security-admin/src/main/webapp/WEB-INF/log4j.properties 08ac852 
  security-admin/src/main/webapp/WEB-INF/log4j.xml f7d40bb 
  storm-agent/src/main/java/org/apache/ranger/services/storm/client/StormClient.java 432ed78 

Diff: https://reviews.apache.org/r/47065/diff/


Testing
-------

Verified that the logs are going to renamed file.


Thanks,

Gautam Borad


Re: Review Request 47065: RANGER-975 : Rename ranger log file from xa_portal.log to ranger_admin.log

Posted by Ramesh Mani <rm...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47065/#review132071
-----------------------------------------------------------


Ship it!




Ship It!

- Ramesh Mani


On May 6, 2016, 4:59 p.m., Gautam Borad wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47065/
> -----------------------------------------------------------
> 
> (Updated May 6, 2016, 4:59 p.m.)
> 
> 
> Review request for ranger, Don Bosco Durai, Abhay Kulkarni, Madhan Neethiraj, Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-975
>     https://issues.apache.org/jira/browse/RANGER-975
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> **Problem Statement:**
> xa_portal.log is a very old name and needs to be renamed to ranger_admin for all log files. 
> 
> 
> **Proposed Solution: **
> 1) In ranger_admin dir> ews/logs/  the logs xa_portal.logs has been renamed to ranger_admin.log. 
> 2) Same as ranger_admin.log the changes are done for ranger_admin_sql_db_patch.log and ranger_admin_sql.log.
> 3) On plugin side pop-up has been changed with text: ".....Check ranger_admin.log for more info".
> 
> 
> Diffs
> -----
> 
>   agents-common/src/main/java/org/apache/ranger/plugin/client/BaseClient.java 86a91a4 
>   hbase-agent/src/main/java/org/apache/ranger/services/hbase/client/HBaseClient.java 1979905 
>   hdfs-agent/src/main/java/org/apache/ranger/services/hdfs/client/HdfsClient.java 4d563a3 
>   hive-agent/src/main/java/org/apache/ranger/services/hive/client/HiveClient.java e38c26c 
>   knox-agent/src/main/java/org/apache/ranger/services/knox/client/KnoxClient.java 8af77b2 
>   plugin-kms/src/main/java/org/apache/ranger/services/kms/client/KMSClient.java 271392b 
>   plugin-yarn/src/main/java/org/apache/ranger/services/yarn/client/YarnClient.java 0c7bcad 
>   security-admin/src/main/java/org/apache/ranger/common/TimedExecutor.java ca2e2dc 
>   security-admin/src/main/webapp/WEB-INF/db_patch.log4j.xml 1aafb07 
>   security-admin/src/main/webapp/WEB-INF/log4j.dev.xml d1c8b79 
>   security-admin/src/main/webapp/WEB-INF/log4j.properties 08ac852 
>   security-admin/src/main/webapp/WEB-INF/log4j.xml f7d40bb 
>   storm-agent/src/main/java/org/apache/ranger/services/storm/client/StormClient.java 432ed78 
> 
> Diff: https://reviews.apache.org/r/47065/diff/
> 
> 
> Testing
> -------
> 
> Verified that the logs are going to renamed file.
> 
> 
> Thanks,
> 
> Gautam Borad
> 
>