Segment Replication - Remove unnecessary call to markAllocationIdAsIn…

…Sync. (opensearch-project#4224) This PR Removes an unnecessary call to markAllocationIdAsInSync on the primary shard when replication events complete. Recovery will manage this initial call. Signed-off-by: Marc Handalian <[email protected]> Signed-off-by: Marc Handalian <[email protected]>
Rishikesh1159 · Aug 17, 2022 · 360405e · 360405e
1 parent 9511a89
commit 360405e
Showing 1 changed file with 0 additions and 8 deletions.
diff --git a/server/src/main/java/org/opensearch/indices/replication/SegmentReplicationSourceHandler.java b/server/src/main/java/org/opensearch/indices/replication/SegmentReplicationSourceHandler.java
@@ -147,14 +147,6 @@ public synchronized void sendFiles(GetSegmentFilesRequest request, ActionListene
             transfer.start();
 
             sendFileStep.whenComplete(r -> {
-                final String targetAllocationId = request.getTargetAllocationId();
-                RunUnderPrimaryPermit.run(
-                    () -> shard.markAllocationIdAsInSync(targetAllocationId, request.getCheckpoint().getSeqNo()),
-                    shard.shardId() + " marking " + targetAllocationId + " as in sync",
-                    shard,
-                    cancellableThreads,
-                    logger
-                );
                 try {
                     future.onResponse(new GetSegmentFilesResponse(List.of(storeFileMetadata)));
                 } finally {