-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52839/
-----------------------------------------------------------
(Updated Oct. 19, 2016, 10:51 p.m.)
Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, Nate Cole, and
Sid Wagle.
Changes
-------
Hi,
Implemented most of the review comments.
Have attached final code before shipping it.
Regards,
Amar
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
