From 8d3632caf85b69a306b7f7b50095425eb177eb77 Mon Sep 17 00:00:00 2001
From: Gunnar Beutner <gunnar.beutner@netways.de>
Date: Tue, 24 Jun 2014 11:46:53 +0200
Subject: [PATCH] Cache parent and child object for dependencies

fixes #6565
---
 lib/icinga/dependency.cpp | 57 +++++++++++++++++++++------------------
 lib/icinga/dependency.hpp |  3 +++
 2 files changed, 34 insertions(+), 26 deletions(-)

diff --git a/lib/icinga/dependency.cpp b/lib/icinga/dependency.cpp
index e8d906e16..fc9dff210 100644
--- a/lib/icinga/dependency.cpp
+++ b/lib/icinga/dependency.cpp
@@ -62,15 +62,39 @@ void Dependency::OnStateLoaded(void)
 
 	ASSERT(!OwnsLock());
 
-	if (!GetChild())
+	Host::Ptr childHost = Host::GetByName(GetChildHostName());
+
+	if (childHost) {
+		if (GetChildServiceName().IsEmpty()) {
+			Log(LogDebug, "Dependency", "Dependency '" + GetName() + "' child host '" + GetChildHostName() + ".");
+			m_Child = childHost;
+		} else {
+			Log(LogDebug, "Dependency", "Dependency '" + GetName() + "' child host '" + GetChildHostName() + "' service '" + GetChildServiceName() + "' .");
+			m_Child = childHost->GetServiceByShortName(GetChildServiceName());
+		}
+	}
+	
+	if (!m_Child)
 		Log(LogWarning, "Dependency", "Dependency '" + GetName() + "' references an invalid child object and will be ignored.");
 	else
-		GetChild()->AddDependency(GetSelf());
-
-	if (!GetParent())
+		m_Child->AddDependency(GetSelf());
+
+	Host::Ptr parentHost = Host::GetByName(GetParentHostName());
+
+	if (parentHost) {
+		if (GetParentServiceName().IsEmpty()) {
+			Log(LogDebug, "Dependency", "Dependency '" + GetName() + "' parent host '" + GetParentHostName() + ".");
+			m_Parent = parentHost;
+		} else {
+			Log(LogDebug, "Dependency", "Dependency '" + GetName() + "' parent host '" + GetParentHostName() + "' service '" + GetParentServiceName() + "' .");
+			m_Parent = parentHost->GetServiceByShortName(GetParentServiceName());
+		}
+	}
+	
+	if (!m_Parent)
 		Log(LogWarning, "Dependency", "Dependency '" + GetName() + "' references an invalid parent object and will always fail.");
 	else
-		GetParent()->AddReverseDependency(GetSelf());
+		m_Parent->AddReverseDependency(GetSelf());
 }
 
 void Dependency::Stop(void)
@@ -149,31 +173,12 @@ bool Dependency::IsAvailable(DependencyType dt) const
 
 Checkable::Ptr Dependency::GetChild(void) const
 {
-	Host::Ptr host = Host::GetByName(GetChildHostName());
-
-	if (!host)
-		return Service::Ptr();
-
-	if (GetChildServiceName().IsEmpty())
-		return host;
-	else
-		return host->GetServiceByShortName(GetChildServiceName());
+	return m_Child;
 }
 
 Checkable::Ptr Dependency::GetParent(void) const
 {
-	Host::Ptr host = Host::GetByName(GetParentHostName());
-
-	if (!host)
-		return Service::Ptr();
-
-	if (GetParentServiceName().IsEmpty()) {
-		Log(LogDebug, "Dependency", "Dependency '" + GetName() + "' parent host '" + GetParentHostName() + ".");
-		return host;
-	} else {
-		Log(LogDebug, "Dependency", "Dependency '" + GetName() + "' parent host '" + GetParentHostName() + "' service '" + GetParentServiceName() + "' .");
-		return host->GetServiceByShortName(GetParentServiceName());
-	}
+	return m_Parent;
 }
 
 TimePeriod::Ptr Dependency::GetPeriod(void) const
diff --git a/lib/icinga/dependency.hpp b/lib/icinga/dependency.hpp
index 4d310c956..08e04e813 100644
--- a/lib/icinga/dependency.hpp
+++ b/lib/icinga/dependency.hpp
@@ -57,6 +57,9 @@ protected:
 	virtual void Stop(void);
 
 private:
+	Checkable::Ptr m_Parent;
+	Checkable::Ptr m_Child;
+
 	static bool EvaluateApplyRuleOne(const Checkable::Ptr& checkable, const ApplyRule& rule);
 	static void EvaluateApplyRule(const ApplyRule& rule);
 	static void EvaluateApplyRules(const std::vector<ApplyRule>& rules);
-- 
2.40.0