aboutsummaryrefslogtreecommitdiffhomepage
path: root/patches/server/0605-Time-scoreboard-search.patch
diff options
context:
space:
mode:
Diffstat (limited to 'patches/server/0605-Time-scoreboard-search.patch')
-rw-r--r--patches/server/0605-Time-scoreboard-search.patch43
1 files changed, 43 insertions, 0 deletions
diff --git a/patches/server/0605-Time-scoreboard-search.patch b/patches/server/0605-Time-scoreboard-search.patch
new file mode 100644
index 0000000000..8d15084758
--- /dev/null
+++ b/patches/server/0605-Time-scoreboard-search.patch
@@ -0,0 +1,43 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: Spottedleaf <[email protected]>
+Date: Tue, 21 Apr 2020 01:53:22 -0700
+Subject: [PATCH] Time scoreboard search
+
+Plugins leaking scoreboards will make this very expensive,
+let server owners debug it easily
+
+diff --git a/src/main/java/co/aikar/timings/MinecraftTimings.java b/src/main/java/co/aikar/timings/MinecraftTimings.java
+index e2764186bd6b838ed5cd86c15597a08d079ef984..6b3cde6d4d1e63bec01f502f2027ee9fddac08aa 100644
+--- a/src/main/java/co/aikar/timings/MinecraftTimings.java
++++ b/src/main/java/co/aikar/timings/MinecraftTimings.java
+@@ -46,6 +46,7 @@ public final class MinecraftTimings {
+
+ public static final Timing antiXrayUpdateTimer = Timings.ofSafe("anti-xray - update");
+ public static final Timing antiXrayObfuscateTimer = Timings.ofSafe("anti-xray - obfuscate");
++ public static final Timing scoreboardScoreSearch = Timings.ofSafe("Scoreboard score search"); // Paper - add timings for scoreboard search
+
+ private static final Map<Class<?>, String> taskNameCache = new MapMaker().weakKeys().makeMap();
+
+diff --git a/src/main/java/org/bukkit/craftbukkit/scoreboard/CraftScoreboardManager.java b/src/main/java/org/bukkit/craftbukkit/scoreboard/CraftScoreboardManager.java
+index c7ca6210d6ae37fe95068c9baa5fb654f95307e0..cad42a0f3c016bf65181e50d139ae4e2fb9158a5 100644
+--- a/src/main/java/org/bukkit/craftbukkit/scoreboard/CraftScoreboardManager.java
++++ b/src/main/java/org/bukkit/craftbukkit/scoreboard/CraftScoreboardManager.java
+@@ -113,9 +113,18 @@ public final class CraftScoreboardManager implements ScoreboardManager {
+
+ // CraftBukkit method
+ public void forAllObjectives(ObjectiveCriteria criteria, ScoreHolder holder, Consumer<ScoreAccess> consumer) {
++ // Paper start - add timings for scoreboard search
++ // plugins leaking scoreboards will make this very expensive, let server owners debug it easily
++ co.aikar.timings.MinecraftTimings.scoreboardScoreSearch.startTimingIfSync();
++ try {
++ // Paper end - add timings for scoreboard search
+ for (CraftScoreboard scoreboard : this.scoreboards) {
+ Scoreboard board = scoreboard.board;
+ board.forAllObjectives(criteria, holder, (score) -> consumer.accept(score));
+ }
++ } finally { // Paper start - add timings for scoreboard search
++ co.aikar.timings.MinecraftTimings.scoreboardScoreSearch.stopTimingIfSync();
++ }
++ // Paper end - add timings for scoreboard search
+ }
+ }