protect FileStore.deadChunks against race codition

andreitokar · andreitokar · commit 57d233227ee4 · 2023-08-22T09:40:33.000-04:00
diff --git a/h2/src/main/org/h2/mvstore/FileStore.java b/h2/src/main/org/h2/mvstore/FileStore.java
@@ -14,7 +14,6 @@
 import java.io.IOException;
 import java.nio.ByteBuffer;
 import java.nio.channels.FileChannel;
-import java.util.ArrayDeque;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.BitSet;
@@ -33,6 +32,7 @@
 import java.util.TreeMap;
 import java.util.concurrent.ArrayBlockingQueue;
 import java.util.concurrent.ConcurrentHashMap;
+import java.util.concurrent.ConcurrentLinkedDeque;
 import java.util.concurrent.ConcurrentMap;
 import java.util.concurrent.ExecutionException;
 import java.util.concurrent.Future;
@@ -194,7 +194,7 @@ public abstract class FileStore<C extends Chunk<C>>
      */
     private MVMap<String, String> layout;
 
-    private final Deque<C> deadChunks = new ArrayDeque<>();
+    private final Deque<C> deadChunks = new ConcurrentLinkedDeque<>();
 
     /**
      * Reference to a background thread, which is expected to be running, if any.
