-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/33743/
-----------------------------------------------------------
Review request for Ambari, Dmitro Lisnichenko, Jonathan Hurley, Sumit Mohanty,
and Sid Wagle.
Bugs: AMBARI-10877
https://issues.apache.org/jira/browse/AMBARI-10877
Repository: ambari
Description
-------
Delete a host (even if request does not specify a cluster name), should delete
all references to it.
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/HostResourceProvider.java
45900e4
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/HostConfigMappingDAO.java
34d0e3c
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/HostRoleCommandDAO.java
7d3f4e4
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/HostStateDAO.java
f939de3
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/HostVersionDAO.java
de3b8cb
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/ServiceConfigDAO.java
8f8e196
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostRoleCommandEntity.java
3919555
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostStateEntity.java
52ae322
ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java
3764dd1
ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClustersImpl.java
9e63ff2
ambari-server/src/main/java/org/apache/ambari/server/state/host/HostImpl.java
27f4800
ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerTest.java
ffe35af
ambari-server/src/test/java/org/apache/ambari/server/orm/dao/HostConfigMappingDAOTest.java
0dcc471
ambari-server/src/test/java/org/apache/ambari/server/orm/dao/ServiceConfigDAOTest.java
3d93e4d
Diff: https://reviews.apache.org/r/33743/diff/
Testing
-------
Deployed a cluster with several hosts, and performed more advanced actions like
creating config groups and overrides to test the deletion.
Local unit tests passed
Thanks,
Alejandro Fernandez
