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

(Updated Dec. 22, 2016, 10:24 p.m.)


Review request for Ambari and Sumit Mohanty.


Changes
-------

Incorporated review comments


Bugs: AMBARI-19274
    https://issues.apache.org/jira/browse/AMBARI-19274


Repository: ambari


Description
-------

Add common log4j settings to advanced config - hbase, zookeeper, ams


Diffs (updated)
-----

  
ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/configuration/ams-hbase-log4j.xml
 08c7b94 
  
ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/configuration/ams-log4j.xml
 81f6dbc 
  
ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/ams.py
 1ad3b85 
  
ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/hbase.py
 7fb9715 
  
ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/params.py
 ad66ffe 
  
ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/configuration/hbase-log4j.xml
 8495de1 
  
ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/hbase.py
 e513677 
  
ambari-server/src/main/resources/common-services/HBASE/0.96.0.2.0/package/scripts/params_linux.py
 8200a85 
  
ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/configuration/yarn-log4j.xml
 89dd52d 
  
ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5/configuration/zookeeper-log4j.xml
 04f872c 
  
ambari-server/src/main/resources/common-services/ZOOKEEPER/3.4.5/package/scripts/params_linux.py
 84cc431 
  
ambari-server/src/main/resources/stacks/HDP/2.3/services/YARN/configuration/yarn-log4j.xml
 84c6db1 

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


Testing
-------

1. Ensure that log4j rolling parameters appear in UI as separate properties
2. Change value and check if reflected in Database, API and file


Thanks,

Madhuvanthi Radhakrishnan

Reply via email to