----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47456/#review133634 -----------------------------------------------------------
ambari-server/src/main/resources/scripts/takeover_files_mapping.json (line 2) <https://reviews.apache.org/r/47456/#comment198187> If we are creating a mapping file for file/path=>config-type, we should add other commonly used paths and map them to config-types. Files like 'log4j.properties' which maps to hdfs-log4j, yarn-log4j, etc... This file by default should have all the most common paths, and their correct config-types. - Srimanth Gunturi On May 17, 2016, 2:16 p.m., Andrew Onischuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47456/ > ----------------------------------------------------------- > > (Updated May 17, 2016, 2:16 p.m.) > > > Review request for Ambari, Sid Wagle and Vitalyi Brodetskyi. > > > Bugs: AMBARI-16703 > https://issues.apache.org/jira/browse/AMBARI-16703 > > > Repository: ambari > > > Description > ------- > > hbase-site conflicts between cluster and AMS. > > Hard code the config path as /etc/ams-hbase/conf to be AMS HBase configs. > > > Diffs > ----- > > ambari-server/src/main/resources/scripts/takeover_config_merge.py f940f27 > ambari-server/src/main/resources/scripts/takeover_files_mapping.json > PRE-CREATION > > Diff: https://reviews.apache.org/r/47456/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Andrew Onischuk > >
