From 1b852010568dc3b21c0c296e2f0a2d6ff0b8bed3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Oliver=20Gond=C5=BEa?= <ogondza@gmail.com>
Date: Tue, 29 Sep 2015 21:20:03 +0200
Subject: [PATCH] [FIXED JENKINS-30705] Optimize TagCloud calculation

---
 core/src/main/java/hudson/model/Label.java | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/core/src/main/java/hudson/model/Label.java b/core/src/main/java/hudson/model/Label.java
index f0a2df2474..30c3905c53 100644
--- a/core/src/main/java/hudson/model/Label.java
+++ b/core/src/main/java/hudson/model/Label.java
@@ -85,6 +85,7 @@ public abstract class Label extends Actionable implements Comparable<Label>, Mod
     protected transient final String name;
     private transient volatile Set<Node> nodes;
     private transient volatile Set<Cloud> clouds;
+    private transient volatile int tiedJobsCount;
 
     @Exported
     public transient final LoadStatistics loadStatistics;
@@ -376,6 +377,8 @@ public abstract class Label extends Actionable implements Comparable<Label>, Mod
      * @return a count of projects that are tied on this node.
      */
     public int getTiedJobCount() {
+        if (tiedJobsCount != -1) return tiedJobsCount;
+
         // denormalize for performance
         // we don't need to respect security as much when returning a simple count
         SecurityContext context = ACL.impersonate(ACL.SYSTEM);
@@ -412,7 +415,7 @@ public abstract class Label extends Actionable implements Comparable<Label>, Mod
                     }
                 }
             }
-            return result;
+            return tiedJobsCount = result;
         } finally {
             SecurityContextHolder.setContext(context);
         }
@@ -433,6 +436,7 @@ public abstract class Label extends Actionable implements Comparable<Label>, Mod
     /*package*/ void reset() {
         nodes = null;
         clouds = null;
+        tiedJobsCount = -1;
     }
 
     /**
-- 
GitLab