Remove redundant null check

Signed-off-by: Bhumika Saini <[email protected]>
opensearch-project · Apr 25, 2024 · d28f6da · d28f6da
1 parent 9be0001
commit d28f6da
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/server/src/main/java/org/opensearch/index/seqno/ReplicationTracker.java b/server/src/main/java/org/opensearch/index/seqno/ReplicationTracker.java
@@ -1108,10 +1108,9 @@ private ReplicationGroup calculateReplicationGroup() {
         }
         assert newVersion == 0 || indexSettings.isRemoteTranslogStoreEnabled()
         // Handle migration cases. Ignore assertion if any of the shard copies in the replication group is assigned to a remote node
-            || (replicationGroup != null
-                && replicationGroup.getReplicationTargets()
-                    .stream()
-                    .anyMatch(shardRouting -> isShardOnRemoteEnabledNode.apply(shardRouting.currentNodeId())))
+            || replicationGroup.getReplicationTargets()
+                .stream()
+                .anyMatch(shardRouting -> isShardOnRemoteEnabledNode.apply(shardRouting.currentNodeId()))
             || checkpoints.entrySet().stream().filter(e -> e.getValue().tracked).allMatch(e -> e.getValue().replicated)
             : "In absence of remote translog store, all tracked shards must have replication mode as LOGICAL_REPLICATION";