From 9b8fe85edb81378f16d3f0819b4f5604bad7e7cf Mon Sep 17 00:00:00 2001
From: Jason Tedor <jason@tedor.me>
Date: Wed, 12 Sep 2018 14:38:24 -0400
Subject: [PATCH] Remove volatile from global checkpoint listeners (#33636)

This field does not need to be volatile because all accesses are done
under a lock. This commit removes the unnecessary volatile modifier from
this field.
---
 .../elasticsearch/index/shard/GlobalCheckpointListeners.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/server/src/main/java/org/elasticsearch/index/shard/GlobalCheckpointListeners.java b/server/src/main/java/org/elasticsearch/index/shard/GlobalCheckpointListeners.java
index df93a935b62..224d5be17e1 100644
--- a/server/src/main/java/org/elasticsearch/index/shard/GlobalCheckpointListeners.java
+++ b/server/src/main/java/org/elasticsearch/index/shard/GlobalCheckpointListeners.java
@@ -63,7 +63,7 @@ public class GlobalCheckpointListeners implements Closeable {
 
     // guarded by this
     private boolean closed;
-    private volatile Map<GlobalCheckpointListener, ScheduledFuture<?>> listeners;
+    private Map<GlobalCheckpointListener, ScheduledFuture<?>> listeners;
     private long lastKnownGlobalCheckpoint = UNASSIGNED_SEQ_NO;
 
     private final ShardId shardId;