----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50860/#review145096 -----------------------------------------------------------
Ship it! Ship It! - Dmitro Lisnichenko On Aug. 5, 2016, 8:17 p.m., Vitalyi Brodetskyi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50860/ > ----------------------------------------------------------- > > (Updated Aug. 5, 2016, 8:17 p.m.) > > > Review request for Ambari, Andrew Onischuk, Dmytro Sen, Sumit Mohanty, and > Sid Wagle. > > > Bugs: AMBARI-18048 > https://issues.apache.org/jira/browse/AMBARI-18048 > > > Repository: ambari > > > Description > ------- > > ambari-server upgrade should print all the config properties that got > modified due to upgrade. Perhaps it can generate a report file. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java > fae9378 > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/AbstractUpgradeCatalog.java > 57cce3f > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/SchemaUpgradeHelper.java > ebfb267 > > ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog.java > 41c2215 > > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeTest.java > a1fd32d > > Diff: https://reviews.apache.org/r/50860/diff/ > > > Testing > ------- > > mvn clean test > > For Unit tests i will create separate jira. Because ConfigurationTest calss > is brocken.!!!!!!!!!!!!!!! > > > Thanks, > > Vitalyi Brodetskyi > >