aboutsummaryrefslogtreecommitdiffhomepage
path: root/Spigot-Server-Patches/0158-Auto-Save-Improvements.patch
diff options
context:
space:
mode:
authorZach Brown <[email protected]>2017-03-25 00:22:02 -0500
committerZach Brown <[email protected]>2017-03-25 00:22:02 -0500
commit5e2784e941ba2873facc44b30be906485fcdae95 (patch)
treebd89c6f56ded2bfabe6ffe468f15bc7ce329a1b9 /Spigot-Server-Patches/0158-Auto-Save-Improvements.patch
parent246712ed92d21dbc90f9e8437dc42c79b558308d (diff)
downloadPaper-5e2784e941ba2873facc44b30be906485fcdae95.tar.gz
Paper-5e2784e941ba2873facc44b30be906485fcdae95.zip
Switch to a bStats based Metrics system
Diffstat (limited to 'Spigot-Server-Patches/0158-Auto-Save-Improvements.patch')
-rw-r--r--Spigot-Server-Patches/0158-Auto-Save-Improvements.patch219
1 files changed, 0 insertions, 219 deletions
diff --git a/Spigot-Server-Patches/0158-Auto-Save-Improvements.patch b/Spigot-Server-Patches/0158-Auto-Save-Improvements.patch
deleted file mode 100644
index 3f6dd66f2c..0000000000
--- a/Spigot-Server-Patches/0158-Auto-Save-Improvements.patch
+++ /dev/null
@@ -1,219 +0,0 @@
-From 3a0addf1653cf2821958eeccea104b889a887798 Mon Sep 17 00:00:00 2001
-From: Aikar <[email protected]>
-Date: Mon, 19 Sep 2016 23:16:39 -0400
-Subject: [PATCH] Auto Save Improvements
-
-Makes Auto Save Rate setting configurable per-world. If the auto save rate is left -1, the global bukkit.yml value will be used.
-
-Process auto save every tick instead of once per auto tick interval, so that chunk saves will distribute over many ticks instead of all at once.
-
-Re-introduce a cap per tick for auto save (Spigot disabled the vanilla cap) and make it configurable.
-
-Adds incremental player auto saving too
-
-diff --git a/src/main/java/com/destroystokyo/paper/PaperConfig.java b/src/main/java/com/destroystokyo/paper/PaperConfig.java
-index cc0ae9224..17102d95d 100644
---- a/src/main/java/com/destroystokyo/paper/PaperConfig.java
-+++ b/src/main/java/com/destroystokyo/paper/PaperConfig.java
-@@ -250,4 +250,9 @@ public class PaperConfig {
- flyingKickPlayerMessage = getString("messages.kick.flying-player", flyingKickPlayerMessage);
- flyingKickVehicleMessage = getString("messages.kick.flying-vehicle", flyingKickVehicleMessage);
- }
-+
-+ public static int playerAutoSaveRate = -1;
-+ private static void playerAutoSaveRate() {
-+ playerAutoSaveRate = getInt("settings.player-auto-save-rate", -1);
-+ }
- }
-diff --git a/src/main/java/com/destroystokyo/paper/PaperWorldConfig.java b/src/main/java/com/destroystokyo/paper/PaperWorldConfig.java
-index dc96bd7f4..bcf24cb49 100644
---- a/src/main/java/com/destroystokyo/paper/PaperWorldConfig.java
-+++ b/src/main/java/com/destroystokyo/paper/PaperWorldConfig.java
-@@ -2,6 +2,7 @@ package com.destroystokyo.paper;
-
- import java.util.List;
-
-+import net.minecraft.server.MinecraftServer;
- import org.bukkit.Bukkit;
- import org.bukkit.configuration.file.YamlConfiguration;
- import org.spigotmc.SpigotWorldConfig;
-@@ -359,4 +360,19 @@ public class PaperWorldConfig {
- private void elytraHitWallDamage() {
- elytraHitWallDamage = getBoolean("elytra-hit-wall-damage", true);
- }
-+
-+ public int autoSavePeriod = -1;
-+ private void autoSavePeriod() {
-+ autoSavePeriod = getInt("auto-save-interval", -1);
-+ if (autoSavePeriod > 0) {
-+ log("Auto Save Interval: " +autoSavePeriod + " (" + (autoSavePeriod / 20) + "s)");
-+ } else if (autoSavePeriod < 0) {
-+ autoSavePeriod = MinecraftServer.getServer().autosavePeriod;
-+ }
-+ }
-+
-+ public int maxAutoSaveChunksPerTick = 24;
-+ private void maxAutoSaveChunksPerTick() {
-+ maxAutoSaveChunksPerTick = getInt("max-auto-save-chunks-per-tick", 24);
-+ }
- }
-diff --git a/src/main/java/net/minecraft/server/Chunk.java b/src/main/java/net/minecraft/server/Chunk.java
-index 88437d77a..9f7f32dc2 100644
---- a/src/main/java/net/minecraft/server/Chunk.java
-+++ b/src/main/java/net/minecraft/server/Chunk.java
-@@ -960,11 +960,9 @@ public class Chunk {
- if (this.t && this.world.getTime() != this.lastSaved || this.s) {
- return true;
- }
-- } else if (this.t && this.world.getTime() >= this.lastSaved + MinecraftServer.getServer().autosavePeriod * 4) { // Spigot - Only save if we've passed 2 auto save intervals without modification
-- return true;
- }
--
-- return this.s;
-+ // This !flag section should say if s(isModified) or t(hasEntities), then check auto save
-+ return ((this.s || this.t) && this.world.getTime() >= this.lastSaved + world.paperConfig.autoSavePeriod); // Paper - Make world configurable and incremental
- }
-
- public Random a(long i) {
-diff --git a/src/main/java/net/minecraft/server/ChunkProviderServer.java b/src/main/java/net/minecraft/server/ChunkProviderServer.java
-index 0a8e09e5e..ad668be62 100644
---- a/src/main/java/net/minecraft/server/ChunkProviderServer.java
-+++ b/src/main/java/net/minecraft/server/ChunkProviderServer.java
-@@ -1,5 +1,6 @@
- package net.minecraft.server;
-
-+import com.destroystokyo.paper.PaperConfig;
- import com.google.common.collect.Lists;
- import com.google.common.collect.Sets;
- import it.unimi.dsi.fastutil.longs.Long2ObjectMap;
-@@ -266,7 +267,7 @@ public class ChunkProviderServer implements IChunkProvider {
- this.saveChunk(chunk);
- chunk.f(false);
- ++i;
-- if (i == 24 && !flag && false) { // Spigot
-+ if (!flag && i >= world.paperConfig.maxAutoSaveChunksPerTick) { // Spigot - // Paper - Incremental Auto Save - cap max per tick
- return false;
- }
- }
-diff --git a/src/main/java/net/minecraft/server/EntityPlayer.java b/src/main/java/net/minecraft/server/EntityPlayer.java
-index 93c4babd9..0cd350254 100644
---- a/src/main/java/net/minecraft/server/EntityPlayer.java
-+++ b/src/main/java/net/minecraft/server/EntityPlayer.java
-@@ -30,6 +30,7 @@ public class EntityPlayer extends EntityHuman implements ICrafting {
-
- private static final Logger bR = LogManager.getLogger();
- public String locale = null; // Spigot private -> public // Paper - default to null
-+ public long lastSave = MinecraftServer.currentTick; // Paper
- public PlayerConnection playerConnection;
- public final MinecraftServer server;
- public final PlayerInteractManager playerInteractManager;
-diff --git a/src/main/java/net/minecraft/server/MinecraftServer.java b/src/main/java/net/minecraft/server/MinecraftServer.java
-index f83cc7641..d49a4ded0 100644
---- a/src/main/java/net/minecraft/server/MinecraftServer.java
-+++ b/src/main/java/net/minecraft/server/MinecraftServer.java
-@@ -118,6 +118,7 @@ public abstract class MinecraftServer implements Runnable, ICommandListener, IAs
- public final Thread primaryThread;
- public java.util.Queue<Runnable> processQueue = new java.util.concurrent.ConcurrentLinkedQueue<Runnable>();
- public int autosavePeriod;
-+ public boolean serverAutoSave = false; // Paper
- // CraftBukkit end
- // Spigot start
- public final SlackActivityAccountant slackActivityAccountant = new SlackActivityAccountant();
-@@ -765,22 +766,30 @@ public abstract class MinecraftServer implements Runnable, ICommandListener, IAs
- this.q.b().a(agameprofile);
- }
-
-- if (autosavePeriod > 0 && this.ticks % autosavePeriod == 0) { // CraftBukkit
- this.methodProfiler.a("save");
-- this.v.savePlayers();
-+
-+ serverAutoSave = (autosavePeriod > 0 && this.ticks % autosavePeriod == 0); // Paper
-+ int playerSaveInterval = com.destroystokyo.paper.PaperConfig.playerAutoSaveRate;
-+ if (playerSaveInterval < 0) {
-+ playerSaveInterval = autosavePeriod;
-+ }
-+ if (playerSaveInterval > 0) { // CraftBukkit // Paper
-+ this.v.savePlayers(playerSaveInterval);
- // Spigot Start
-+ } // Paper - Incremental Auto Saving
-+
- // We replace this with saving each individual world as this.saveChunks(...) is broken,
- // and causes the main thread to sleep for random amounts of time depending on chunk activity
- // Also pass flag to only save modified chunks
- server.playerCommandState = true;
- for (World world : worlds) {
-- world.getWorld().save(false);
-+ if (world.paperConfig.autoSavePeriod > 0) world.getWorld().save(false); // Paper - Incremental / Configurable Auto Saving
- }
- server.playerCommandState = false;
- // this.saveChunks(true);
- // Spigot End
- this.methodProfiler.b();
-- }
-+ //} // Paper - Incremental Auto Saving
-
- this.methodProfiler.a("tallying");
- // Spigot start
-diff --git a/src/main/java/net/minecraft/server/PlayerList.java b/src/main/java/net/minecraft/server/PlayerList.java
-index be67df061..f675a7013 100644
---- a/src/main/java/net/minecraft/server/PlayerList.java
-+++ b/src/main/java/net/minecraft/server/PlayerList.java
-@@ -330,6 +330,7 @@ public abstract class PlayerList {
- }
-
- protected void savePlayerFile(EntityPlayer entityplayer) {
-+ entityplayer.lastSave = MinecraftServer.currentTick; // Paper
- this.playerFileData.save(entityplayer);
- ServerStatisticManager serverstatisticmanager = (ServerStatisticManager) this.o.get(entityplayer.getUniqueID());
-
-@@ -1205,13 +1206,23 @@ public abstract class PlayerList {
-
- }
-
-+ // Paper start
- public void savePlayers() {
-+ savePlayers(null);
-+ }
-+
-+ public void savePlayers(Integer interval) {
-+ long now = MinecraftServer.currentTick;
- MinecraftTimings.savePlayers.startTiming(); // Paper
- for (int i = 0; i < this.players.size(); ++i) {
-- this.savePlayerFile((EntityPlayer) this.players.get(i));
-+ EntityPlayer entityplayer = this.players.get(i);
-+ if (interval == null || now - entityplayer.lastSave >= interval) {
-+ this.savePlayerFile(entityplayer);
-+ }
- }
- MinecraftTimings.savePlayers.stopTiming(); // Paper
- }
-+ // Paper end
-
- public void addWhitelist(GameProfile gameprofile) {
- this.whitelist.add(new WhiteListEntry(gameprofile));
-diff --git a/src/main/java/net/minecraft/server/WorldServer.java b/src/main/java/net/minecraft/server/WorldServer.java
-index f969d2a72..cc0e8d2c8 100644
---- a/src/main/java/net/minecraft/server/WorldServer.java
-+++ b/src/main/java/net/minecraft/server/WorldServer.java
-@@ -1003,8 +1003,9 @@ public class WorldServer extends World implements IAsyncTaskHandler {
- ChunkProviderServer chunkproviderserver = this.getChunkProviderServer();
-
- if (chunkproviderserver.e()) {
-- org.bukkit.Bukkit.getPluginManager().callEvent(new org.bukkit.event.world.WorldSaveEvent(getWorld())); // CraftBukkit
-+ if (flag) org.bukkit.Bukkit.getPluginManager().callEvent(new org.bukkit.event.world.WorldSaveEvent(getWorld())); // CraftBukkit // Paper - Incremental Auto Saving - Only fire event on full save
- timings.worldSave.startTiming(); // Paper
-+ if (flag || server.serverAutoSave) { // Paper
- if (iprogressupdate != null) {
- iprogressupdate.a("Saving level");
- }
-@@ -1013,6 +1014,7 @@ public class WorldServer extends World implements IAsyncTaskHandler {
- if (iprogressupdate != null) {
- iprogressupdate.c("Saving chunks");
- }
-+ } // Paper
-
- timings.worldSaveChunks.startTiming(); // Paper
- chunkproviderserver.a(flag);
---
-2.12.0.windows.1
-