No need for mutex, just read shardRouting once

elastic · May 30, 2018 · 1d4e044 · 1d4e044
1 parent 903ef15
commit 1d4e044
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/server/src/main/java/org/elasticsearch/index/shard/IndexShard.java b/server/src/main/java/org/elasticsearch/index/shard/IndexShard.java
@@ -2066,9 +2066,8 @@ public void startRecovery(RecoveryState recoveryState, PeerRecoveryTargetService
     }
 
     public ShardStateMetaData getShardStateMetaData() {
-        synchronized (mutex) {
-            return new ShardStateMetaData(shardRouting.primary(), indexSettings.getUUID(), shardRouting.allocationId());
-        }
+        final ShardRouting shardRouting = this.shardRouting;
+        return new ShardStateMetaData(shardRouting.primary(), indexSettings.getUUID(), shardRouting.allocationId());
     }
 
     /**