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

(Updated April 15, 2016, 4:03 p.m.)


Review request for Ambari, Jayush Luniya, Robert Nettleton, and Sumit Mohanty.


Changes
-------

- added xml support (xml validtion)


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


Repository: ambari


Description
-------

Reading config values (like xml files or *.sh scripts) from external property 
files.

I did not added xml support for reading external property files. Reason is 
because these xml files are jinja templates. So they can act as text files.

also added: Solr Instance renamed to LogSearch Solr Instance


Diffs (updated)
-----

  
ambari-server/src/main/java/org/apache/ambari/server/stack/ConfigurationDirectory.java
 84842e2 
  ambari-server/src/main/java/org/apache/ambari/server/utils/XmlUtils.java 
PRE-CREATION 
  
ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logfeeder-env.xml
 94ed27d 
  
ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logfeeder-log4j.xml
 91fc0ce 
  
ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logsearch-audit_logs-solrconfig.xml
 4da16b1 
  
ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logsearch-env.xml
 ef6bebf 
  
ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logsearch-log4j.xml
 6741f41 
  
ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logsearch-service_logs-solrconfig.xml
 60363c8 
  
ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logsearch-solr-env.xml
 855f0e5 
  
ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logsearch-solr-log4j.xml
 e99cb81 
  
ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logsearch-solr-xml.xml
 d31d0b3 
  ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/metainfo.xml 
018398f 
  
ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/properties/audit_logs-solrconfig.xml.j2
 PRE-CREATION 
  
ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/properties/logfeeder-env.sh.j2
 PRE-CREATION 
  
ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/properties/logfeeder-log4j.xml.j2
 PRE-CREATION 
  
ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/properties/logsearch-env.sh.j2
 PRE-CREATION 
  
ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/properties/logsearch-log4j.xml.j2
 PRE-CREATION 
  
ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/properties/logsearch-solr-env.sh.j2
 PRE-CREATION 
  
ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/properties/service_logs-solrconfig.xml.j2
 PRE-CREATION 
  
ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/properties/solr-log4j.properties.j2
 PRE-CREATION 
  
ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/properties/solr.xml.j2
 PRE-CREATION 

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


Testing
-------

Total run:959
Total errors:0
Total failures:0


Thanks,

Oliver Szabo

Reply via email to