-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37984/
-----------------------------------------------------------

(Updated Sept. 2, 2015, 3:54 a.m.)


Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, Jonathan 
Hurley, and Nate Cole.


Changes
-------

Verified it worked on an upgrade from 2.1.1 to 2.1.2


Bugs: AMBARI-12699
    https://issues.apache.org/jira/browse/AMBARI-12699


Repository: ambari


Description
-------

Make required database schema changes such as moving the upgrade_pack column 
from the repo_version to the upgrade table.
Also, added upgrade_type column to the upgrade_table.

In the process, I changed the UpgradePack class so that it contains a name, and 
changed several methods that expected Map<String, UpgradePack> to 
Collection<UpgradePack>


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/checks/HostsMasterMaintenanceCheck.java
 ef93337 
  
ambari-server/src/main/java/org/apache/ambari/server/checks/SecondaryNamenodeDeletedCheck.java
 493042f 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/PrereqCheckRequest.java
 f8c5316 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/CompatibleRepositoryVersionResourceProvider.java
 6344aa2 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/PreUpgradeCheckResourceProvider.java
 c394498 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RepositoryVersionResourceProvider.java
 f1fa3bf 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
 fa743be 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/spi/Resource.java
 1b208fb 
  
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/RepositoryVersionDAO.java
 4ac1314 
  ambari-server/src/main/java/org/apache/ambari/server/orm/dao/UpgradeDAO.java 
bc0652c 
  
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/RepositoryVersionEntity.java
 0fb2f10 
  
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/UpgradeEntity.java
 802ea03 
  
ambari-server/src/main/java/org/apache/ambari/server/stack/StackDirectory.java 
89c10c6 
  ambari-server/src/main/java/org/apache/ambari/server/state/StackInfo.java 
2aa89cc 
  ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java 
3e25d01 
  
ambari-server/src/main/java/org/apache/ambari/server/state/stack/UpgradePack.java
 79b8eb5 
  
ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/RepositoryVersionHelper.java
 2e17cf4 
  
ambari-server/src/main/java/org/apache/ambari/server/state/stack/upgrade/UpgradeType.java
 17ee22c 
  
ambari-server/src/main/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostImpl.java
 63f015b 
  
ambari-server/src/main/java/org/apache/ambari/server/upgrade/SchemaUpgradeHelper.java
 77e2e93 
  
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog212.java
 PRE-CREATION 
  
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog220.java
 4eb7a80 
  ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql 265e42e 
  ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 0053837 
  ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql 30b669d 
  ambari-server/src/main/resources/Ambari-DDL-Postgres-EMBEDDED-CREATE.sql 
4f7569c 
  ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql 81d0e6f 
  
ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/nonrolling-upgrade-2.2.xml
 bf237c6 
  ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.2.xml 
9b7848f 
  ambari-server/src/main/resources/stacks/HDP/2.2/upgrades/upgrade-2.3.xml 
25df73a 
  ambari-server/src/main/resources/stacks/HDP/2.3/upgrades/upgrade-2.3.xml 
e9671e3 
  
ambari-server/src/test/java/org/apache/ambari/server/checks/HostsMasterMaintenanceCheckTest.java
 fc0f9bc 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/CompatibleRepositoryVersionResourceProviderTest.java
 ea6e56e 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/RepositoryVersionResourceProviderTest.java
 442bcb2 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UpgradeResourceProviderHDP22Test.java
 166b665 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/UpgradeResourceProviderTest.java
 7d2c117 
  ambari-server/src/test/java/org/apache/ambari/server/orm/OrmTestHelper.java 
53f7432 
  ambari-server/src/test/java/org/apache/ambari/server/orm/dao/CrudDAOTest.java 
8777d33 
  
ambari-server/src/test/java/org/apache/ambari/server/orm/dao/RepositoryVersionDAOTest.java
 adda018 
  
ambari-server/src/test/java/org/apache/ambari/server/orm/dao/UpgradeDAOTest.java
 0b12e97 
  
ambari-server/src/test/java/org/apache/ambari/server/serveraction/upgrades/ConfigureActionTest.java
 93e29b5 
  
ambari-server/src/test/java/org/apache/ambari/server/serveraction/upgrades/UpgradeActionTest.java
 91a2788 
  
ambari-server/src/test/java/org/apache/ambari/server/state/UpgradeHelperTest.java
 f7898ee 
  
ambari-server/src/test/java/org/apache/ambari/server/state/stack/UpgradePackTest.java
 a73775f 
  
ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog212Test.java
 PRE-CREATION 
  
ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_bucket_test.xml
 8c2c205 
  
ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_direction.xml
 693c5ec 
  ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_test.xml 
e1dde1b 
  
ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_test_checks.xml
 84d7a5a 
  
ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_test_nonrolling.xml
 a22e68b 
  
ambari-server/src/test/resources/stacks/HDP/2.1.1/upgrades/upgrade_to_new_stack.xml
 fa2a00e 
  ambari-server/src/test/resources/stacks/HDP/2.2.0/upgrades/upgrade_test.xml 
96dd8f7 
  
ambari-server/src/test/resources/stacks/HDP/2.2.0/upgrades/upgrade_test_checks.xml
 099bbd7 

Diff: https://reviews.apache.org/r/37984/diff/


Testing
-------

** This is not a fully functional change yet. The UI has to change in order to 
pass the upgrade_type (ROLLING|NONROLLING) to upgrades endpoint.
I still need to test UpgradeCatalog212, and check what happens when a customer 
upgraded to Ambari 2.1, and needs to absorb the 2.1.2 schema changes.


Thanks,

Alejandro Fernandez

Reply via email to