----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61833/#review183581 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/state/configgroup/ConfigGroupImpl.java Lines 118 (patched) <https://reviews.apache.org/r/61833/#comment259636> Can we pass in service name to the constructor and used that instead of tag? Similar to how this has been done in AMBARI-21078 ? - Sebastian Toader On Aug. 23, 2017, 3:51 a.m., Sid Wagle wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61833/ > ----------------------------------------------------------- > > (Updated Aug. 23, 2017, 3:51 a.m.) > > > Review request for Ambari, Myroslav Papirkovskyy, Sumit Mohanty, and > Sebastian Toader. > > > 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 > ----- > > > 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/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/state/ServiceTest.java > 1ad8cef > > > Diff: https://reviews.apache.org/r/61833/diff/1/ > > > Testing > ------- > > Manually verfied with a 2.5.2 cluster. > Unit test added. > > > Thanks, > > Sid Wagle > >
