----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62249/#review185227 -----------------------------------------------------------
Ship it! Ship It! - Swapan Shridhar On Sept. 12, 2017, 4:47 p.m., Vitalyi Brodetskyi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62249/ > ----------------------------------------------------------- > > (Updated Sept. 12, 2017, 4:47 p.m.) > > > Review request for Ambari, Jayush Luniya and Swapan Shridhar. > > > Bugs: AMBARI-21936 > https://issues.apache.org/jira/browse/AMBARI-21936 > > > Repository: ambari > > > Description > ------- > > yarn-env.sh exports YARN_CONF_DIR > {code:title=yarn-env.sh} > # resolve links - $0 may be a softlink > export YARN_CONF_DIR="${YARN_CONF_DIR:-$HADOOP_YARN_HOME/conf}" > {code} > > This causes in not respecting --config option for yarn daemon. > Hence please remove the export > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/YARN/3.0.0.3.0/configuration/yarn-env.xml > aaa72d1 > > > Diff: https://reviews.apache.org/r/62249/diff/2/ > > > Testing > ------- > > TODO > > > Thanks, > > Vitalyi Brodetskyi > >