----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46207/#review128901 -----------------------------------------------------------
ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/yarn.py (line 76) <https://reviews.apache.org/r/46207/#comment192346> The condition check for starts with "/tmp" and then it creates folder using hdfs_tmp_dir. It looks a bit odd. - Sumit Mohanty On April 14, 2016, 3:07 p.m., Andrew Onischuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46207/ > ----------------------------------------------------------- > > (Updated April 14, 2016, 3:07 p.m.) > > > Review request for Ambari, Sumit Mohanty and Vitalyi Brodetskyi. > > > Bugs: AMBARI-15885 > https://issues.apache.org/jira/browse/AMBARI-15885 > > > Repository: ambari > > > Description > ------- > > . > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/params_linux.py > 91fdb83 > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/yarn.py > bb2f567 > ambari-server/src/test/python/stacks/2.0.6/YARN/test_historyserver.py > 080bc11 > > Diff: https://reviews.apache.org/r/46207/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Andrew Onischuk > >