AMBARI-19679. Dangling config-group hostmapping entries result in partial loading of config groups. (swagle)
Project: http://git-wip-us.apache.org/repos/asf/ambari/repo Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/a51ede84 Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/a51ede84 Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/a51ede84 Branch: refs/heads/branch-dev-patch-upgrade Commit: a51ede8480e22e21a738bfe9921eb49d7e10a3aa Parents: 3ade321 Author: Siddharth Wagle <[email protected]> Authored: Mon Jan 23 11:14:35 2017 -0800 Committer: Siddharth Wagle <[email protected]> Committed: Mon Jan 23 11:14:35 2017 -0800 ---------------------------------------------------------------------- .../apache/ambari/server/state/configgroup/ConfigGroupImpl.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/ambari/blob/a51ede84/ambari-server/src/main/java/org/apache/ambari/server/state/configgroup/ConfigGroupImpl.java ---------------------------------------------------------------------- diff --git a/ambari-server/src/main/java/org/apache/ambari/server/state/configgroup/ConfigGroupImpl.java b/ambari-server/src/main/java/org/apache/ambari/server/state/configgroup/ConfigGroupImpl.java index fe1f338..03edcf8 100644 --- a/ambari-server/src/main/java/org/apache/ambari/server/state/configgroup/ConfigGroupImpl.java +++ b/ambari-server/src/main/java/org/apache/ambari/server/state/configgroup/ConfigGroupImpl.java @@ -172,8 +172,9 @@ public class ConfigGroupImpl implements ConfigGroup { if (host != null && hostEntity != null) { m_hosts.put(hostEntity.getHostId(), host); } - } catch (AmbariException e) { - LOG.warn("Host seems to be deleted but Config group mapping still exists !"); + } catch (Exception e) { + LOG.warn("Host {} seems to be deleted but Config group {} mapping " + + "still exists !", hostMappingEntity.getHostname(), configGroupName); LOG.debug("Host seems to be deleted but Config group mapping still exists !", e); } }
