From 226d31e9cf47b642d4ecd79e642dac94bcaf7277 Mon Sep 17 00:00:00 2001 From: SamB440 Date: Sat, 20 Mar 2021 15:23:16 +0000 Subject: [PATCH] Revert "Fix server freeze by running config save async (YamlStorage)" Caused the JVM to crash. This was unsafe. It's better to just let the server freeze (likely due to slow file system), and tell people to use SQLite instead. This reverts commit bb58da00 --- .../rpgregions/managers/data/yml/YamlStorage.java | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/rpgregions/src/main/java/net/islandearth/rpgregions/managers/data/yml/YamlStorage.java b/rpgregions/src/main/java/net/islandearth/rpgregions/managers/data/yml/YamlStorage.java index f634022..5b7358f 100644 --- a/rpgregions/src/main/java/net/islandearth/rpgregions/managers/data/yml/YamlStorage.java +++ b/rpgregions/src/main/java/net/islandearth/rpgregions/managers/data/yml/YamlStorage.java @@ -5,7 +5,6 @@ import net.islandearth.rpgregions.managers.data.IStorageManager; import net.islandearth.rpgregions.managers.data.account.RPGRegionsAccount; import net.islandearth.rpgregions.managers.data.region.Discovery; import net.islandearth.rpgregions.managers.data.region.WorldDiscovery; -import org.bukkit.Bukkit; import org.bukkit.configuration.file.FileConfiguration; import org.bukkit.configuration.file.YamlConfiguration; @@ -146,13 +145,11 @@ public class YamlStorage implements IStorageManager { } config.set("Discoveries", newData); - Bukkit.getScheduler().runTaskAsynchronously(plugin, () -> { - try { - config.save(file); - } catch (IOException e) { - e.printStackTrace(); - } - }); + try { + config.save(file); + } catch (IOException e) { + e.printStackTrace(); + } cachedAccounts.remove(uuid); } }