----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23549/#review47866 -----------------------------------------------------------
Ship it! Ship It! - Myroslav Papirkovskyy On July 16, 2014, 2:34 p.m., Andrew Onischuk wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/23549/ > ----------------------------------------------------------- > > (Updated July 16, 2014, 2:34 p.m.) > > > Review request for Ambari and Myroslav Papirkovskyy. > > > Bugs: AMBARI-6500 > https://issues.apache.org/jira/browse/AMBARI-6500 > > > Repository: ambari > > > Description > ------- > > If updateIfExists is set to false and there is nothing to update, we should > not create new configs. > > org.apache.ambari.server.upgrade.AbstractUpgradeCatalog#updateConfigurationPro > perties > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/AbstractUpgradeCatalog.java > 2f01d38 > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalogTest.java > 322b3f1 > > Diff: https://reviews.apache.org/r/23549/diff/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Andrew Onischuk > >
