----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25310/#review52449 -----------------------------------------------------------
Ship it! Ship It! - Dmitro Lisnichenko On Sept. 3, 2014, 7:07 p.m., Andrew Onischuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25310/ > ----------------------------------------------------------- > > (Updated Sept. 3, 2014, 7:07 p.m.) > > > Review request for Ambari and Dmitro Lisnichenko. > > > Bugs: AMBARI-7113 > https://issues.apache.org/jira/browse/AMBARI-7113 > > > Repository: ambari > > > Description > ------- > > Precondition: > Deploy cluster with all components and default settings. > > STR: > 1) Navigate on Services->HIVE tab > 2) Click Download Client Configs and download file > 3)Check hive-site.xml files at cluster and in downloaded files. > /etc/hive/conf.dist/hive-site.xml > > Expected result: > Files have same values. Property: javax.jdo.option.ConnectionPassword present > in host (hive-site.xml) and downloaded(hive-site.xml) files. > > Actual result: > Property: javax.jdo.option.ConnectionPassword is **present** in Download > Client Configs files and **absent** in hive-site.xml. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/api/util/StackExtensionHelper.java > c4dd022 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClientConfigResourceProvider.java > 7bec61a > > ambari-server/src/test/java/org/apache/ambari/server/api/util/StackExtensionHelperTest.java > 9a7946a > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClientConfigResourceProviderTest.java > 2f54273 > > ambari-server/src/test/resources/stacks/HDP/2.0.5/services/ZOOKEEPER/metainfo.xml > 2875b77 > > ambari-server/src/test/resources/stacks/HDP/2.0.7/services/ZOOKEEPER/metainfo.xml > d397d19 > > Diff: https://reviews.apache.org/r/25310/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Andrew Onischuk > >
