shouldRollGeneration should execute under read lock (elastic#41696)

Translog#shouldRollGeneration should execute under the read lock since it accesses the current writer.
gurkankaymak · May 27, 2019 · 23b992e · 23b992e
1 parent e8d9d00
commit 23b992e
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/server/src/main/java/org/elasticsearch/index/translog/Translog.java b/server/src/main/java/org/elasticsearch/index/translog/Translog.java
@@ -553,9 +553,10 @@ public Location add(final Operation operation) throws IOException {
      * @return {@code true} if the current generation should be rolled to a new generation
      */
     public boolean shouldRollGeneration() {
-        final long size = this.current.sizeInBytes();
         final long threshold = this.indexSettings.getGenerationThresholdSize().getBytes();
-        return size > threshold;
+        try (ReleasableLock ignored = readLock.acquire()) {
+            return this.current.sizeInBytes() > threshold;
+        }
     }
 
     /**