----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55129/#review160506 -----------------------------------------------------------
Ship it! Ship It! - Nate Cole On Jan. 3, 2017, 3:41 a.m., Jay SenSharma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55129/ > ----------------------------------------------------------- > > (Updated Jan. 3, 2017, 3:41 a.m.) > > > Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, Nate Cole, > and Sid Wagle. > > > Bugs: AMBARI-19337 > https://issues.apache.org/jira/browse/AMBARI-19337 > > > Repository: ambari > > > Description > ------- > > YARN proxyusers properties has spelling mistake. For a long time the property > has ".proxyusers." where as it is supposed to be ".proxyuser." Even though > the incorrect properties will be ignored by YARN however it should be good to > correct the spelling mistake on these properties. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/kerberos.json > a8379ee > > ambari-server/src/main/resources/common-services/YARN/3.0.0.3.0/kerberos.json > 4cb18a9 > ambari-server/src/main/resources/stacks/HDP/2.2/services/YARN/kerberos.json > 784589c > > ambari-server/src/main/resources/stacks/HDP/2.3.ECS/services/YARN/kerberos.json > 74b5746 > ambari-server/src/main/resources/stacks/HDP/2.3/services/YARN/kerberos.json > c20bd23 > ambari-server/src/main/resources/stacks/HDP/2.5/services/YARN/kerberos.json > 4cb18a9 > > ambari-server/src/main/resources/stacks/PERF/1.0/services/YARN/kerberos.json > f33c07d > > ambari-server/src/test/python/stacks/2.2/configs/pig-service-check-secure.json > 0d0c6f5 > > ambari-server/src/test/resources/kerberos/test_kerberos_descriptor_2_1_3.json > a9e0bcd > ambari-web/app/assets/data/stacks/HDP-2.1/service_components.json d853f24 > ambari-web/app/data/configs/wizards/secure_mapping.js 8d952e8 > > Diff: https://reviews.apache.org/r/55129/diff/ > > > Testing > ------- > > Existing testcases does not directly address this. So test not performed. > > > Thanks, > > Jay SenSharma > >
