Skip to content

Commit 66d18a4

Browse files
Abstract out RemoteFsTranslogWithPinnedTimestamps (#15579) (#15623)
* Abstract out RemoteFsTranslog with pinned timestamps (cherry picked from commit deeb2de) Signed-off-by: Sachin Kale <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
1 parent 39aa3e4 commit 66d18a4

File tree

5 files changed

+531
-310
lines changed

5 files changed

+531
-310
lines changed

server/src/main/java/org/opensearch/index/translog/RemoteBlobStoreInternalTranslogFactory.java

Lines changed: 29 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -69,19 +69,35 @@ public Translog newTranslog(
6969

7070
assert repository instanceof BlobStoreRepository : "repository should be instance of BlobStoreRepository";
7171
BlobStoreRepository blobStoreRepository = ((BlobStoreRepository) repository);
72-
return new RemoteFsTranslog(
73-
config,
74-
translogUUID,
75-
deletionPolicy,
76-
globalCheckpointSupplier,
77-
primaryTermSupplier,
78-
persistedSequenceNumberConsumer,
79-
blobStoreRepository,
80-
threadPool,
81-
startedPrimarySupplier,
82-
remoteTranslogTransferTracker,
83-
remoteStoreSettings
84-
);
72+
if (RemoteStoreSettings.isPinnedTimestampsEnabled()) {
73+
return new RemoteFsTimestampAwareTranslog(
74+
config,
75+
translogUUID,
76+
deletionPolicy,
77+
globalCheckpointSupplier,
78+
primaryTermSupplier,
79+
persistedSequenceNumberConsumer,
80+
blobStoreRepository,
81+
threadPool,
82+
startedPrimarySupplier,
83+
remoteTranslogTransferTracker,
84+
remoteStoreSettings
85+
);
86+
} else {
87+
return new RemoteFsTranslog(
88+
config,
89+
translogUUID,
90+
deletionPolicy,
91+
globalCheckpointSupplier,
92+
primaryTermSupplier,
93+
persistedSequenceNumberConsumer,
94+
blobStoreRepository,
95+
threadPool,
96+
startedPrimarySupplier,
97+
remoteTranslogTransferTracker,
98+
remoteStoreSettings
99+
);
100+
}
85101
}
86102

87103
public Repository getRepository() {

0 commit comments

Comments
 (0)