----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46196/#review129297 -----------------------------------------------------------
Ship it! Ship It! - Jayush Luniya On April 15, 2016, 4:03 p.m., Oliver Szabo wrote: > > ----------------------------------------------------------- > 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. > > > 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 > ----- > > > 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 > >