----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57792/#review169706 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java Lines 511-514 (patched) <https://reviews.apache.org/r/57792/#comment242226> Not quite sure what your intent is here. Why change a stage's text like this? ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/ClusterGrouping.java Lines 311-314 (original), 290-312 (patched) <https://reviews.apache.org/r/57792/#comment242225> This code looks like a straight copy/paste out of Grouping.addTasksToStageInBatches(). It would be convenient to have a method in ParallelScheduler itself to do this work or abstract it some other way. ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/StageWrapper.java Lines 40 (patched) <https://reviews.apache.org/r/57792/#comment242230> "...belong in this stage" ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/StageWrapper.java Lines 41-42 (patched) <https://reviews.apache.org/r/57792/#comment242232> Can always formulate the HRC text based on the actual text, I don't see a need for the original text, since you're overwriting it anyway. It's not really the "original text" anymore. ambari-server/src/main/resources/host_scripts/alert_version_select.py Lines 72-73 (patched) <https://reviews.apache.org/r/57792/#comment242222> Nit: %s format is deprecated ambari-server/src/main/resources/host_scripts/alert_version_select.py Lines 82-83 (patched) <https://reviews.apache.org/r/57792/#comment242221> Nit: formatting ambari-server/src/main/resources/upgrade-pack.xsd Line 180 (original), 180-186 (patched) <https://reviews.apache.org/r/57792/#comment242235> This is a copy/paste. Make it it's own type and reference it in both places. ambari-server/src/main/resources/upgrade-pack.xsd Lines 187 (patched) <https://reviews.apache.org/r/57792/#comment242236> The code is one task per ExecuteStage, it's not a list. And it shouldn't be! There is more thought that has to be made before we make ExecuteStage support more than one task. - Nate Cole On March 20, 2017, 8:49 p.m., Alejandro Fernandez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57792/ > ----------------------------------------------------------- > > (Updated March 20, 2017, 8:49 p.m.) > > > Review request for Ambari, Jonathan Hurley, Nate Cole, Sumit Mohanty, and Sid > Wagle. > > > Bugs: AMBARI-20490 > https://issues.apache.org/jira/browse/AMBARI-20490 > > > Repository: ambari > > > Description > ------- > > On large-scale clusters over 1000 nodes, performance during Express Upgrade > can be very slow and it takes more time to fix potential errors. > We can remedy these with the following fixes: > > * "hdp-select set all" is currently ran in the ClusterGroup with a single > stage instead of batching. This can cause Ambari Server to be the bottleneck > when processing 1000+ requests. Allow the execute-stage inside a ClusterGroup > to use batching. > > * If some hosts have symlinks or dirs that don't belong in /usr/hdp/ then it > will prevent the hosts from running "hdp-select versions". We can find these > problematic hosts with an alert. > > * There is currently no upgrade path from HDP 2.2 directly to 2.5. Doing a > 2-step upgrade is very time consuming (only for branch 2.4 since HDP 2.2 no > longer supported in Ambari 2.5). > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/stack_select.py > 20b4cbd > > ambari-server/src/main/java/org/apache/ambari/server/checks/AtlasPresenceCheck.java > 8556436 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java > ca9ce07 > > ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java > ee5451f > > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/ClusterGrouping.java > c45ec24 > > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/Grouping.java > 8003223 > > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/StageWrapper.java > 669d50f > > ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/StageWrapperBuilder.java > 3cf3f8b > ambari-server/src/main/resources/alerts.json 2559b3a > ambari-server/src/main/resources/host_scripts/alert_version_select.py > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/nonrolling-upgrade-2.3.xml > 28ca939 > > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/nonrolling-upgrade-2.4.xml > 9aff1a9 > > ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/nonrolling-upgrade-2.5.xml > adccac0 > > ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/nonrolling-upgrade-2.4.xml > 3c0104b > > ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/nonrolling-upgrade-2.5.xml > cd63651 > > ambari-server/src/main/resources/stacks/HDP/2.5/upgrades/nonrolling-upgrade-2.5.xml > 39cb0dc > > ambari-server/src/main/resources/stacks/HDP/2.5/upgrades/nonrolling-upgrade-2.6.xml > 57a5203 > > ambari-server/src/main/resources/stacks/HDP/2.6/upgrades/nonrolling-upgrade-2.6.xml > 035cb59 > ambari-server/src/main/resources/upgrade-pack.xsd 1f11aa1 > > ambari-server/src/test/java/org/apache/ambari/server/api/services/AmbariMetaInfoTest.java > f1af66f > > ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerTest.java > bf6c41a > > ambari-server/src/test/java/org/apache/ambari/server/metadata/AgentAlertDefinitionsTest.java > cbc5e69 > > ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java > 8e5ad0a > > ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_test_batch.xml > PRE-CREATION > > ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_to_new_stack.xml > fe387f3 > > > Diff: https://reviews.apache.org/r/57792/diff/2/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Alejandro Fernandez > >
