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

Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, Jonathan 
Hurley, Nate Cole, and Sid Wagle.


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


Repository: ambari


Description
-------

As part AMBARI-10167 (Delete a host from Ambari cluster is not clean; fails to 
re-add the same host), need all of the host-related tables to switch from a 
host_name to a host_id.


Diffs
-----

  
ambari-server/src/main/java/org/apache/ambari/server/actionmanager/ActionDBAccessorImpl.java
 7447a2d 
  
ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommand.java
 f37e937 
  
ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommandFactory.java
 1126666 
  
ambari-server/src/main/java/org/apache/ambari/server/actionmanager/HostRoleCommandFactoryImpl.java
 b63adfa 
  ambari-server/src/main/java/org/apache/ambari/server/actionmanager/Stage.java 
51d5e8a 
  
ambari-server/src/main/java/org/apache/ambari/server/actionmanager/StageFactoryImpl.java
 PRE-CREATION 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariManagementControllerImpl.java
 a4ddf14 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/ControllerModule.java
 0c5e04a 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelper.java
 5cd75bb 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProvider.java
 e872fe9 
  
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeResourceProvider.java
 9733eff 
  
ambari-server/src/main/java/org/apache/ambari/server/events/listeners/upgrade/HostVersionOutOfSyncListener.java
 dcc06a7 
  
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/HostRoleCommandDAO.java
 f927197 
  
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostEntity.java
 a811c16 
  
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java
 c9877fb 
  
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostVersionEntity.java
 363e6be 
  
ambari-server/src/main/java/org/apache/ambari/server/stageplanner/RoleGraph.java
 4fe3787 
  
ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
 1a8bf43 
  
ambari-server/src/main/java/org/apache/ambari/server/upgrade/UpgradeCatalog210.java
 884032e 
  ambari-server/src/main/java/org/apache/ambari/server/utils/StageUtils.java 
020dd4b 
  ambari-server/src/main/resources/Ambari-DDL-MySQL-CREATE.sql c3488f2 
  ambari-server/src/main/resources/Ambari-DDL-Oracle-CREATE.sql 0455e9e 
  ambari-server/src/main/resources/Ambari-DDL-Postgres-CREATE.sql 2c381b2 
  ambari-server/src/main/resources/Ambari-DDL-Postgres-EMBEDDED-CREATE.sql 
24762eb 
  ambari-server/src/main/resources/Ambari-DDL-SQLServer-CREATE.sql adf5828 
  
ambari-server/src/test/java/org/apache/ambari/server/actionmanager/ExecutionCommandWrapperTest.java
 d498c97 
  
ambari-server/src/test/java/org/apache/ambari/server/actionmanager/StageTest.java
 cd424d4 
  
ambari-server/src/test/java/org/apache/ambari/server/actionmanager/TestActionDBAccessorImpl.java
 112e1e5 
  
ambari-server/src/test/java/org/apache/ambari/server/actionmanager/TestActionManager.java
 6c5a8a0 
  
ambari-server/src/test/java/org/apache/ambari/server/actionmanager/TestActionScheduler.java
 dd93176 
  
ambari-server/src/test/java/org/apache/ambari/server/actionmanager/TestStage.java
 7ee6045 
  
ambari-server/src/test/java/org/apache/ambari/server/agent/AgentResourceTest.java
 f4d9c63 
  
ambari-server/src/test/java/org/apache/ambari/server/agent/TestHeartbeatHandler.java
 c6e2788 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelperTest.java
 947a76f 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerTest.java
 3e310ff 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/CalculatedStatusTest.java
 d11dae0 
  
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClusterStackVersionResourceProviderTest.java
 1cc75c7 
  ambari-server/src/test/java/org/apache/ambari/server/orm/OrmTestHelper.java 
6041066 
  
ambari-server/src/test/java/org/apache/ambari/server/orm/dao/HostVersionDAOTest.java
 7cf59e9 
  
ambari-server/src/test/java/org/apache/ambari/server/orm/dao/RequestDAOTest.java
 8ca53f7 
  
ambari-server/src/test/java/org/apache/ambari/server/serveraction/ServerActionExecutorTest.java
 580351f 
  
ambari-server/src/test/java/org/apache/ambari/server/serveraction/upgrades/UpgradeActionTest.java
 87dd18b 
  
ambari-server/src/test/java/org/apache/ambari/server/stageplanner/TestStagePlanner.java
 0a381f9 
  
ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClusterTest.java
 66a4ade 
  ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeTest.java 
7455706 
  
ambari-server/src/test/java/org/apache/ambari/server/utils/TestStageUtils.java 
c411985 

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


Testing
-------

Local unit tests passed on branch that was based on an older commit.
After rebasing, some unit tests failed, so will fix those and re-test on a live 
cluster.


Thanks,

Alejandro Fernandez

Reply via email to