----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40433/#review106996 -----------------------------------------------------------
I think we should add some tests - Vitalyi Brodetskyi On Лис. 18, 2015, 8:18 до полудня, Dmytro Sen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40433/ > ----------------------------------------------------------- > > (Updated Лис. 18, 2015, 8:18 до полудня) > > > Review request for Ambari, Andrew Onischuk and Vitalyi Brodetskyi. > > > Bugs: AMBARI-13936 > https://issues.apache.org/jira/browse/AMBARI-13936 > > > Repository: ambari > > > Description > ------- > > Starting HDP-2.3, Ambari should support ATSv1.5. This involves: > Adding new configs as specified in the doc below. Lets add only the > non-advanced configs to the HDP-2.3 yarn-site (skip all configs after the > section ADVANCED CONFIGS) > Adding one new config to tez-site > Ensure that the DFS paths are created and have right permissions > property: yarn.timeline-service.entity-file-cache-store.active-dir > (FSPermission 01777) > property: yarn.timeline-service.entity-file-cache-store.done-dir > (FSPermission 0700) > Thing(s) to make sure: > yarn.timeline-service.entity-file-cache-store.active-dir and done-dir exist > on the cluster on hdfs. Active dir should have permission 01777, owned by > yarn-user, group hadoop-group. Done dir should have permission 0700, owned by > yarn-user, group hadoop-group. > The Tez cache plugin class is in timeline server’s classpath - it should be > the case even now, but lets check > yarn.timeline-service.leveldb-timeline-store.path is a path on local > filesystem and needs to be created with permissions 755 ( owner being the > same user as the one which timeline server is running as ) - this should be > happening as well > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/params_linux.py > cc42481 > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/yarn.py > aa79fe4 > > ambari-server/src/main/resources/stacks/HDP/2.3/services/TEZ/configuration/tez-site.xml > 37db69f > > ambari-server/src/main/resources/stacks/HDP/2.3/services/YARN/configuration/yarn-site.xml > c9e617b > > Diff: https://reviews.apache.org/r/40433/diff/ > > > Testing > ------- > > Unit tests passed > > > Thanks, > > Dmytro Sen > >
