Caching the 'parent' value, which resulted in

a drastic performance increase.

However, profiling has shown that GRATR just isn't going
to cut it.  I think I'm going to have to replace it as my
graphing base class to avoid the horrible, horrible performance
I keep encountering.

1 files changed, 15 insertions, 12 deletions

diff --git a/lib/puppet/type.rb b/lib/puppet/type.rb
index 0a6ccdaa4..b7ff1f664 100644
--- a/lib/puppet/type.rb
+++ b/lib/puppet/type.rb
@@ -316,19 +316,22 @@ class Type
     def parent
         return nil unless configuration
 
-        # This is kinda weird.
-        if implicit?
-            parents = configuration.relationship_graph.adjacent(self, :direction => :in)
-        else
-            parents = configuration.adjacent(self, :direction => :in)
-        end
-        if parents
-            # We should never have more than one parent, so let's just ignore
-            # it if we happen to.
-            return parents.shift
-        else
-            return nil
+        unless defined?(@parent)
+            # This is kinda weird.
+            if implicit?
+                parents = configuration.relationship_graph.adjacent(self, :direction => :in)
+            else
+                parents = configuration.adjacent(self, :direction => :in)
+            end
+            if parents
+                # We should never have more than one parent, so let's just ignore
+                # it if we happen to.
+                @parent = parents.shift
+            else
+                @parent = nil
+            end
         end
+        @parent
     end
 
     # Return the "type[name]" style reference.