> On Aug. 11, 2017, 7:41 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/yarn.py > > Lines 330 (patched) > > <https://reviews.apache.org/r/61592/diff/1/?file=1795532#file1795532line330> > > > > Also change this in HDP 3.0
Thanks for the reminder, but HDP 3.0 is [already OK](https://github.com/apache/ambari/blob/trunk/ambari-server/src/main/resources/common-services/YARN/3.0.0.3.0/package/scripts/yarn.py#L326-L333). - Attila ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61592/#review182723 ----------------------------------------------------------- On Aug. 11, 2017, 11:07 a.m., Attila Doroszlai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61592/ > ----------------------------------------------------------- > > (Updated Aug. 11, 2017, 11:07 a.m.) > > > Review request for Ambari, Andrew Onischuk, Sebastian Toader, and Vitalyi > Brodetskyi. > > > Bugs: AMBARI-21708 > https://issues.apache.org/jira/browse/AMBARI-21708 > > > Repository: ambari > > > Description > ------- > > Restore recursive permissions for `mapreduce_jobhistory_done_dir`, remove for > `node_labels_dir` as intended in > [AMBARI-21116](https://issues.apache.org/jira/browse/AMBARI-21116). > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/yarn.py > 28d14fed3baa136551f8099defe7e6884bb20027 > ambari-server/src/test/python/stacks/2.0.6/YARN/test_historyserver.py > ba3d0abb7ef918abb0fd9f199fed27b4b5b41b22 > > > Diff: https://reviews.apache.org/r/61592/diff/1/ > > > Testing > ------- > > Tested History Server manually on local cluster. > > Unit tests: > > ``` > $ mvn -am -pl ambari-server -DskipSurefireTests clean test > ... > Total run:1145 > Total errors:0 > Total failures:0 > OK > ... > [INFO] BUILD SUCCESS > ``` > > > Thanks, > > Attila Doroszlai > >
