----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49386/#review140021 -----------------------------------------------------------
Ship it! Thanks Nate! - Alejandro Fernandez On June 29, 2016, 5:19 p.m., Nate Cole wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/49386/ > ----------------------------------------------------------- > > (Updated June 29, 2016, 5:19 p.m.) > > > Review request for Ambari, Alejandro Fernandez and Jonathan Hurley. > > > Bugs: AMBARI-17485 > https://issues.apache.org/jira/browse/AMBARI-17485 > > > Repository: ambari > > > Description > ------- > > This is just a case of a bad merge. branch-2.4 did not bring over the > "import os" line for a couple of files. > > However, trunk also has the "has_attr" bug, so that will get fixed there. > Also added tests. > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/script/script.py > 77228a9 > > ambari-server/src/main/resources/common-services/SPARK/1.2.1/package/scripts/spark_client.py > bade06e > > ambari-server/src/main/resources/common-services/TEZ/0.4.0.2.1/package/scripts/tez.py > 01940d7 > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/mapreduce2_client.py > 7b166ba > ambari-server/src/test/python/stacks/2.0.6/YARN/test_mapreduce2_client.py > 0083d75 > ambari-server/src/test/python/stacks/2.1/TEZ/test_tez_client.py 73615a4 > ambari-server/src/test/python/stacks/2.2/SPARK/test_spark_client.py a8e580a > > Diff: https://reviews.apache.org/r/49386/diff/ > > > Testing > ------- > > Manual. Automated: > > ---------------------------------------------------------------------- > Total run:1089 > Total errors:0 > Total failures:0 > OK > > > Thanks, > > Nate Cole > >