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

(Updated Oct. 18, 2016, 6:29 p.m.)


Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, Nate Cole, and 
Sid Wagle.


Changes
-------

implemented minor comments and attached diff


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


Repository: ambari


Description
-------

Ability to enable the DEBUG logs in Ambari-server without server restart

1. PropertyConfigurator would check log4j.properties update every 5mins. if 
there is any change in the file/timestamp it will reload the log4j 
configurations.
2. have defined to LOG4JMONITOR_DELAY for 5mins - 300000 milliseconds
3. Have modified only for ambari-server
4.


Diffs (updated)
-----

  
ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java
 378db18 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariServer.java
 c05840e 

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


Testing
-------

1. Change the log4j.properties from INFO to DEBUG
[Expected Result] server should start logging the DEBUG logs with in 5mins of 
time.
[Resutl] PASS

2. Change the log4j.properties from DEBUG to INFO
[Expected Result] server should reduce the logging to INFO mode with in 5mins 
of time.
[Resutl] PASS


Thanks,

Amarnath reddy pappu

Reply via email to