----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33743/#review82244 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/HostConfigMappingDAO.java <https://reviews.apache.org/r/33743/#comment132940> hostConfigMappingByHost is not a thread-safe collection. Concurrent threads editing it will probably cause a ConcurrentModException ambari-server/src/main/java/org/apache/ambari/server/orm/dao/HostConfigMappingDAO.java <https://reviews.apache.org/r/33743/#comment132939> Any reason this can't be a NamedQuery? ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostStateEntity.java <https://reviews.apache.org/r/33743/#comment132941> TODO's are fine (I actually prefer `!!! TODO: ` as the format), but I don't think your name needs to be in it :) - Jonathan Hurley On April 30, 2015, 9:31 p.m., Alejandro Fernandez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33743/ > ----------------------------------------------------------- > > (Updated April 30, 2015, 9:31 p.m.) > > > 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 > >
