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

(Updated Aug. 16, 2016, 4:19 p.m.)


Review request for Ambari, Alejandro Fernandez and Sumit Mohanty.


Changes
-------

Fixed review comments.   Tested in a ambari installed cluster that new Falcon 
service installation allows log4j.xml to be configurable via Ambari


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


Repository: ambari


Description
-------

We should have falcon log4j.xml configurable via ambari


Diffs (updated)
-----

  
ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/configuration/falcon-log4j.xml
 PRE-CREATION 
  
ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/metainfo.xml 
0c49209 

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


Testing
-------

Ran all unit tests and passed


----------------------------------------------------------------------
Ran 267 tests in 6.637s

OK
----------------------------------------------------------------------
Total run:1120
Total errors:0
Total failures:0
OK


Thanks,

Venkat Ranganathan

Reply via email to