----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33489/#review81396 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/FinalizeUpgradeAction.java <https://reviews.apache.org/r/33489/#comment131737> This is a minor thing, but my preference is to have separate methods for error checking and updating DB. - Alejandro Fernandez On April 23, 2015, 7:56 p.m., Nate Cole wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33489/ > ----------------------------------------------------------- > > (Updated April 23, 2015, 7:56 p.m.) > > > Review request for Ambari, Alejandro Fernandez and Jonathan Hurley. > > > Bugs: AMBARI-10005 > https://issues.apache.org/jira/browse/AMBARI-10005 > > > Repository: ambari > > > Description > ------- > > * Cascade the desired stack id to sub-objects: service, component, > host_component > * New configs should only be added on upgrade, not downgrade (separate jira) > * Non-reportable components should be handled in Finalize > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/agent/HeartBeatHandler.java > 596525b > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java > 82b7307 > > ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/FinalizeUpgradeAction.java > d97ad61 > ambari-server/src/main/java/org/apache/ambari/server/state/Cluster.java > 64fe765 > > ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java > f62e3f3 > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UpgradeResourceProviderTest.java > 5de2eac > > Diff: https://reviews.apache.org/r/33489/diff/ > > > Testing > ------- > > Manual 2.2.x -> 2.2.y and 2.2 -> 2.3 + automated: > > Tests run: 2881, Failures: 0, Errors: 0, Skipped: 17 > > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 24:57.558s > [INFO] Finished at: Thu Apr 23 15:49:20 EDT 2015 > [INFO] Final Memory: 40M/714M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Nate Cole > >
