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

Review request for Ambari, DIPAYAN BHOWMICK, Jaimin Jetly, Rohit Choudhary, and 
Yusaku Sako.


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


Repository: ambari


Description
-------

Earlier : 
logs of all the views used to go into ambari-server.log which caused problem in 
debugging views.

In this Patch:
ViewRegistry now looks for file
 view.log4j.properties using the view archive classloader and loads all the
 log4j properties in it using PropertyConfigurator. Separate log4j
 configuraiton files added to all core views.


Diffs
-----

  ambari-server/src/main/java/org/apache/ambari/server/view/ViewRegistry.java 
e31037e 
  contrib/views/capacity-scheduler/pom.xml 5569859 
  contrib/views/capacity-scheduler/src/main/resources/view.log4j.properties 
PRE-CREATION 
  contrib/views/files/pom.xml ccea5ab 
  contrib/views/files/src/main/resources/view.log4j.properties PRE-CREATION 
  contrib/views/hive/pom.xml 1f150ff 
  contrib/views/hive/src/main/resources/view.log4j.properties PRE-CREATION 
  contrib/views/jobs/pom.xml d797ebd 
  contrib/views/jobs/src/main/resources/view.log4j.properties PRE-CREATION 
  contrib/views/pig/pom.xml 97160cc 
  contrib/views/pig/src/main/resources/view.log4j.properties PRE-CREATION 
  contrib/views/slider/pom.xml 608988f 
  contrib/views/slider/src/main/resources/view.log4j.properties PRE-CREATION 
  contrib/views/storm/src/main/resources/view.log4j.properties PRE-CREATION 
  contrib/views/tez/pom.xml 8e82c5e 
  contrib/views/tez/src/main/resources/view.log4j.properties PRE-CREATION 

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


Testing
-------

Manual Testing Done.
separate log files for each view was properly generated. Also the log level was 
getting properly set.


Thanks,

Nitiraj Rathore

Reply via email to