----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61833/ -----------------------------------------------------------
(Updated Aug. 23, 2017, 5:10 p.m.) Review request for Ambari, Myroslav Papirkovskyy, Sumit Mohanty, and Sebastian Toader. Changes ------- Minor nit corrected. Bugs: AMBARI-21784 https://issues.apache.org/jira/browse/AMBARI-21784 Repository: ambari Description ------- Ambari server log is constantly populated with 'Config inconsistency exists: unknown configType=solr-site' even after Solr deletion due to existence of config group. {code} 22 Aug 2017 19:00:20,094 ERROR [ambari-hearbeat-monitor] HostImpl:1085 - Config inconsistency exists: unknown configType=solr-site {code} Diffs (updated) ----- ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ConfigGroupResourceProvider.java 200cf27 ambari-server/src/main/java/org/apache/ambari/server/state/cluster/ClusterImpl.java 3fb45ab ambari-server/src/main/java/org/apache/ambari/server/state/configgroup/ConfigGroupFactory.java 906d948 ambari-server/src/main/java/org/apache/ambari/server/state/configgroup/ConfigGroupImpl.java 9058a63 ambari-server/src/main/java/org/apache/ambari/server/state/host/HostImpl.java 7282485 ambari-server/src/test/java/org/apache/ambari/server/controller/AmbariManagementControllerTest.java 5214f04 ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ConfigGroupResourceProviderTest.java eec379e ambari-server/src/test/java/org/apache/ambari/server/state/ConfigGroupTest.java c70dcc6 ambari-server/src/test/java/org/apache/ambari/server/state/ConfigHelperTest.java 21ce2c2 ambari-server/src/test/java/org/apache/ambari/server/state/ServiceTest.java 1ad8cef ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClusterTest.java e8e7206 ambari-server/src/test/java/org/apache/ambari/server/state/svccomphost/ServiceComponentHostTest.java fc1f334 Diff: https://reviews.apache.org/r/61833/diff/3/ Changes: https://reviews.apache.org/r/61833/diff/2-3/ Testing ------- Manually verfied with a 2.5.2 cluster. Unit test added. Thanks, Sid Wagle