-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31683/
-----------------------------------------------------------
Review request for Ambari, Jonathan Hurley and Tom Beerbower.
Bugs: AMBARI-9897
https://issues.apache.org/jira/browse/AMBARI-9897
Repository: ambari
Description
-------
* Added a "Summary DTO" that will get all the stages' task statuses in one call
to the DB, indexed by stage. This is used instead of iterating, and
re-iterating several times over stages/tasks for a request.
* Most notable, on large clusters we end up with 1000s of stages and tasks.
All that iterating was consuming up to 2 minutes for the full
upgrade/upgrade_group/upgrade_item/task tree. It's now down to about 5s (not
great, but I'll take it).
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/CalculatedStatus.java
3a84529
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/StageResourceProvider.java
e55897c
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeGroupResourceProvider.java
a0ff3d9
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
9a56e2b
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/HostRoleCommandDAO.java
a55f383
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/HostRoleCommandStatusSummaryDTO.java
PRE-CREATION
ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java
76df049
ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/ColocatedGrouping.java
6252d4e
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/StageResourceProviderTest.java
1d80f47
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UpgradeResourceProviderTest.java
ef4bd82
Diff: https://reviews.apache.org/r/31683/diff/
Testing
-------
Manual testing of cluster install + upgrade to make sure both RU-related and
non-RU-related tasks showed the correct numbers.
JUnit results pending.
Thanks,
Nate Cole
