-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37682/
-----------------------------------------------------------
(Updated Sept. 9, 2015, 1:58 p.m.)
Review request for Ambari, Alejandro Fernandez, Dmytro Grinenko, Jonathan
Hurley, and Nate Cole.
Changes
-------
Fixed comments, refactored all configure tasks in 2.2->2.2+, 2.2->2.3,
2.3->2.3+ upgrade packs to a new style. Going to commit patch if it looks good.
Created jiras for adding config change for SWU, and for fixing ignored unit
tests after implementing cross-stack upgrade
Summary (updated)
-----------------
Stop-and-Start Upgrade: Move Configs out of Upgrade Pack
Bugs: AMBARI-12700
https://issues.apache.org/jira/browse/AMBARI-12700
Repository: ambari
Description
-------
The configs need to move out of the Upgrade Packs and into their own file.
This will make it easier to maintain, and clearer since there will not be any
dups.
Since it is going to be a massive change, it would be great to get early
feedback. Code is not complete (still full of TODOs and does not even build)
Diffs (updated)
-----
ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java
4afa9b0
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
dddec73
ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/ConfigureAction.java
c717582
ambari-server/src/main/java/org/apache/ambari/server/stack/ModuleFileUnmarshaller.java
aa8e17b
ambari-server/src/main/java/org/apache/ambari/server/stack/StackDefinitionDirectory.java
8f81b5a
ambari-server/src/main/java/org/apache/ambari/server/stack/StackDirectory.java
db947ca
ambari-server/src/main/java/org/apache/ambari/server/stack/StackModule.java
4b88aff
ambari-server/src/main/java/org/apache/ambari/server/state/StackInfo.java
87301e5
ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java
ecefe6e
ambari-server/src/main/java/org/apache/ambari/server/state/stack/ConfigUpgradePatch.java
PRE-CREATION
ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/ConfigureTask.java
8361ea6
ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/ConfigureTask.java
8361ea6
ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/Grouping.java
9d89b7a
ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/StageWrapperBuilder.java
c9c6b8c
ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/config-upgrade.xml
PRE-CREATION
ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.2.xml
74eb499
ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.3.xml
7c1a1f9
ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.3.xml
7c1a1f9
ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.3.xml
044c43a
ambari-server/src/test/java/org/apache/ambari/server/serveraction/upgrades/ConfigureActionTest.java
e702e0a
ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java
2eee2df
ambari-server/src/test/java/org/apache/ambari/server/state/stack/UpgradePackTest.java
fc731d9
Diff: https://reviews.apache.org/r/37682/diff/
Testing (updated)
-------
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Ambari Views ...................................... SUCCESS [2.992s]
[INFO] Ambari Metrics Common ............................. SUCCESS [1.675s]
[INFO] Ambari Server ..................................... SUCCESS [54:25.287s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 54:30.573s
[INFO] Finished at: Tue Sep 08 18:53:06 EEST 2015
[INFO] Final Memory: 60M/1072M
Thanks,
Dmitro Lisnichenko
