fix mem leak

common/src/main/java/com/dfsek/terra/config/base/PluginConfig.java

@@ -38,13 +38,14 @@ public class PluginConfig implements ConfigTemplate {
     @Value("cache.carver")
     @Default
     private int carverCache = 512;
+
     @Value("cache.structure")
     @Default
     private int structureCache = 128;
 
-    @Value("cache.checks")
+    @Value("cache.sampler")
     @Default
-    private int checkCache = 128;
+    private int samplerCache = 512;
 
     @Value("dump-default")
     @Default
@@ -95,7 +96,7 @@ public class PluginConfig implements ConfigTemplate {
         return structureCache;
     }
 
-    public int getCheckCache() {
+    public int getSamplerCache() {
-        return checkCache;
+        return samplerCache;
     }
 }

common/src/main/java/com/dfsek/terra/generation/math/SamplerCache.java

@@ -5,45 +5,42 @@ import com.dfsek.terra.api.platform.TerraPlugin;
 import com.dfsek.terra.api.platform.world.World;
 import net.jafama.FastMath;
 
+import java.util.Collections;
 import java.util.HashMap;
 import java.util.LinkedHashMap;
 import java.util.Map;
 
 public class SamplerCache {
-    private final Map<Long, Container> cache;
+    private final Map<Long, Container> containerMap;
     private final TerraPlugin main;
 
     public SamplerCache(TerraPlugin main) {
-        cache = new HashMap<>();
+        containerMap = new HashMap<>();
         this.main = main;
     }
 
     public Sampler get(World world, int x, int z) {
-        return cache.computeIfAbsent(world.getSeed(), seed -> new Container(world, new LinkedHashMap<Long, Sampler>() {
+        return containerMap.computeIfAbsent(world.getSeed(), seed -> new Container(world)).get(x, z);
-            @Override
-            protected boolean removeEldestEntry(Map.Entry<Long, Sampler> eldest) {
-                return size() > main.getTerraConfig().getCheckCache();
-            }
-        })).get(x, z);
     }
 
     public Sampler getChunk(World world, int chunkX, int chunkZ) {
-        return cache.computeIfAbsent(world.getSeed(), seed -> new Container(world, new LinkedHashMap<Long, Sampler>() {
+        return containerMap.computeIfAbsent(world.getSeed(), seed -> new Container(world)).getChunk(chunkX, chunkZ);
-            @Override
-            protected boolean removeEldestEntry(Map.Entry<Long, Sampler> eldest) {
-                return size() > main.getTerraConfig().getCheckCache();
-            }
-        })).getChunk(chunkX, chunkZ);
     }
 
 
     private class Container {
         private final World world;
-        private final Map<Long, Sampler> cache;
+        private final TerraWorld terraWorld;
+        private final Map<Long, Sampler> cache = Collections.synchronizedMap(new LinkedHashMap<Long, Sampler>() {
+            @Override
+            protected boolean removeEldestEntry(Map.Entry<Long, Sampler> eldest) {
+                return this.size() > main.getTerraConfig().getSamplerCache();
+            }
+        });
 
-        private Container(World world, Map<Long, Sampler> cache) {
+        private Container(World world) {
             this.world = world;
-            this.cache = cache;
+            terraWorld = main.getWorld(world);
         }
 
         public Sampler get(int x, int z) {
@@ -54,8 +51,9 @@ public class SamplerCache {
 
         public Sampler getChunk(int cx, int cz) {
             long key = (((long) cx) << 32) | (cz & 0xffffffffL);
-            TerraWorld tw = main.getWorld(world);
+            synchronized(cache) {
-            return cache.computeIfAbsent(key, k -> new Sampler(cx, cz, tw.getGrid(), world, tw.getConfig().getTemplate().getBaseBlend(), tw.getConfig().getTemplate().getElevationBlend()));
+                return cache.computeIfAbsent(key, k -> new Sampler(cx, cz, terraWorld.getGrid(), world, terraWorld.getConfig().getTemplate().getBaseBlend(), terraWorld.getConfig().getTemplate().getElevationBlend()));
+            }
         }
     }
 }

platforms/bukkit/src/main/resources/config.yml

@@ -6,5 +6,6 @@ biome-search-resolution: 4
 cache:
   carver: 512
   structure: 128
+  sampler: 512
 master-disable:
   caves: false