Skip to content

Commit

Permalink
added support for incremental download of translog files
Browse files Browse the repository at this point in the history
Presently, the download workflow for remote backed storage works in a manner which causes the download for same translog files multiple times, each time deleting all the older files before downloading them again. This causes significant wasted network bandwidth, along with the time taken for the shard to become active.

This change adds support for downloading the translog files incrementally and omitting the same if they are present locally.

Signed-off-by: Harsh Rawat <[email protected]>
  • Loading branch information
rawahars committed Oct 7, 2024
1 parent 146b0f7 commit f17a378
Show file tree
Hide file tree
Showing 11 changed files with 337 additions and 34 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -254,14 +254,68 @@ private static void downloadOnce(
Files.createDirectories(location);
}

// Delete translog files on local before downloading from remote
Map<String, String> generationToPrimaryTermMapper = translogMetadata.getGenerationToPrimaryTermMapper();
Map<String, String> generationToChecksumMapper = translogMetadata.getGenerationToChecksumMapper();
long maxGeneration = translogMetadata.getGeneration();
long minGeneration = translogMetadata.getMinTranslogGeneration();

// Delete any translog and checkpoint file which is not part of the current generation range.
for (Path file : FileSystemUtils.files(location)) {
Files.delete(file);
try {
long generation = parseIdFromFileName(file.getFileName().toString(), STRICT_TLOG_OR_CKP_PATTERN);
if (generation < minGeneration || generation > maxGeneration) {
// If the generation is outside the required range, then we delete the same.
Files.delete(file);
}
} catch (IllegalStateException | IllegalArgumentException e) {
// Delete any file which does not conform to Translog or Checkpoint filename patterns.
Files.delete(file);
}
}

Map<String, String> generationToPrimaryTermMapper = translogMetadata.getGenerationToPrimaryTermMapper();
for (long i = translogMetadata.getGeneration(); i >= translogMetadata.getMinTranslogGeneration(); i--) {
for (long i = maxGeneration; i >= minGeneration; i--) {
String generation = Long.toString(i);

// For incremental downloads, we will check if the local translog matches the one present in
// remote store. If so, we will skip its download.
String translogFilename = Translog.getFilename(i);
Path targetTranslogPath = location.resolve(translogFilename);

// If we have the translog available for the generation then we need to
// compare the checksum with that in remote obtained via metadata.
// For backward compatibility, we consider the following cases here-
// - Remote metadata does not have the mapping for generation
// - Local translog file lacks the checksum value in footer
// In both these cases, we will download the files for the generation.
if (generationToChecksumMapper.containsKey(generation) && FileSystemUtils.exists(targetTranslogPath)) {
try {
final long expectedChecksum = Long.parseLong(generationToChecksumMapper.get(generation));
final Long actualChecksum = TranslogFooter.readChecksum(targetTranslogPath);

// If the local and remote checksum are same, then continue.
// Else exit the loop and download the translog.
if (actualChecksum != null && actualChecksum == expectedChecksum) {
logger.info(
"Download skipped for translog and checkpoint files for generation={} due to them being locally present",
generation
);

// Mark the translog and checkpoint file as downloaded in the file tracker.
translogTransferManager.markFileAsDownloaded(translogFilename);
translogTransferManager.markFileAsDownloaded(Translog.getCommitCheckpointFileName(i));
continue;
}
} catch (IOException e) {

Check warning on line 308 in server/src/main/java/org/opensearch/index/translog/RemoteFsTranslog.java

View check run for this annotation

Codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/RemoteFsTranslog.java#L308

Added line #L308 was not covered by tests
// The exception can occur if the remote translog files were uploaded without footer.
logger.info(

Check warning on line 310 in server/src/main/java/org/opensearch/index/translog/RemoteFsTranslog.java

View check run for this annotation

Codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/RemoteFsTranslog.java#L310

Added line #L310 was not covered by tests
"Exception occurred during reconciliation of translog state between local and remote. "
+ "Reverting to downloading the translog and checksum files for generation={}",
generation
);
}

Check warning on line 315 in server/src/main/java/org/opensearch/index/translog/RemoteFsTranslog.java

View check run for this annotation

Codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/RemoteFsTranslog.java#L315

Added line #L315 was not covered by tests
}

logger.info("Downloading translog and checkpoint files for generation={}", generation);
translogTransferManager.downloadTranslog(generationToPrimaryTermMapper.get(generation), generation, location);
}
logger.info(
Expand Down
11 changes: 8 additions & 3 deletions server/src/main/java/org/opensearch/index/translog/Translog.java
Original file line number Diff line number Diff line change
Expand Up @@ -134,6 +134,7 @@ public abstract class Translog extends AbstractIndexShardComponent implements In
public static final String CHECKPOINT_SUFFIX = ".ckp";
public static final String CHECKPOINT_FILE_NAME = "translog" + CHECKPOINT_SUFFIX;

static final Pattern STRICT_TLOG_OR_CKP_PATTERN = Pattern.compile("^" + TRANSLOG_FILE_PREFIX + "(\\d+)(\\.ckp|\\.tlog)$");
static final Pattern PARSE_STRICT_ID_PATTERN = Pattern.compile("^" + TRANSLOG_FILE_PREFIX + "(\\d+)(\\.tlog)$");
public static final int DEFAULT_HEADER_SIZE_IN_BYTES = TranslogHeader.headerSizeInBytes(UUIDs.randomBase64UUID());

Expand Down Expand Up @@ -320,14 +321,18 @@ public static long parseIdFromFileName(Path translogFile) {
return parseIdFromFileName(fileName);
}

public static long parseIdFromFileName(String fileName) {
final Matcher matcher = PARSE_STRICT_ID_PATTERN.matcher(fileName);
public static long parseIdFromFileName(String translogFile) {
return parseIdFromFileName(translogFile, PARSE_STRICT_ID_PATTERN);
}

public static long parseIdFromFileName(String fileName, Pattern pattern) {
final Matcher matcher = pattern.matcher(fileName);
if (matcher.matches()) {
try {
return Long.parseLong(matcher.group(1));
} catch (NumberFormatException e) {
throw new IllegalStateException(
"number formatting issue in a file that passed PARSE_STRICT_ID_PATTERN: " + fileName + "]",
"number formatting issue in a file that passed " + pattern.pattern() + ": " + fileName + "]",

Check warning on line 335 in server/src/main/java/org/opensearch/index/translog/Translog.java

View check run for this annotation

Codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/Translog.java#L335

Added line #L335 was not covered by tests
e
);
}
Expand Down
135 changes: 135 additions & 0 deletions server/src/main/java/org/opensearch/index/translog/TranslogFooter.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,135 @@
/*
* SPDX-License-Identifier: Apache-2.0
*
* The OpenSearch Contributors require contributions made to
* this file be licensed under the Apache-2.0 license or a
* compatible open source license.
*/

/*
* Licensed to Elasticsearch under one or more contributor
* license agreements. See the NOTICE file distributed with
* this work for additional information regarding copyright
* ownership. Elasticsearch licenses this file to you under
* the Apache License, Version 2.0 (the "License"); you may
* not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing,
* software distributed under the License is distributed on an
* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
* KIND, either express or implied. See the License for the
* specific language governing permissions and limitations
* under the License.
*/

/*
* Modifications Copyright OpenSearch Contributors. See
* GitHub history for details.
*/

package org.opensearch.index.translog;

import org.apache.lucene.codecs.CodecUtil;
import org.apache.lucene.store.OutputStreamDataOutput;
import org.opensearch.common.io.Channels;
import org.opensearch.core.common.io.stream.OutputStreamStreamOutput;

import java.io.ByteArrayOutputStream;
import java.io.IOException;
import java.nio.ByteBuffer;
import java.nio.channels.FileChannel;
import java.nio.file.Path;
import java.nio.file.StandardOpenOption;

/**
* Each translog file is started with a header followed by the translog operations, and ending with a footer.
* The footer encapsulates the checksum of the translog.
* */
public class TranslogFooter {

Check warning on line 51 in server/src/main/java/org/opensearch/index/translog/TranslogFooter.java

View check run for this annotation

Codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/TranslogFooter.java#L51

Added line #L51 was not covered by tests

/**
* footerLength returns the length of the footer.
* We are writing 16 bytes and therefore, we return the same.
*/
static int footerLength() {
return 16;
}

/**
* write the translog footer which records both checksum and algorithm ID.
* This method is based upon the CodecUtils.writeFooter method.
* This footer can be parsed and read with TranslogFooter.readChecksum().
*
* Similar to CodecUtils documentation, the footer consists of-
* Footer --> Magic,AlgorithmID,Checksum
* Magic --> Uint32. This identifies the start of the footer. It is always -1071082520.
* AlgorithmID --> Uint32. This indicates the checksum algorithm used. Currently, this is always 0.
* Checksum --> Uint64. This is the checksum as calculated for the translog.
* */
static void write(FileChannel channel, long checksum, boolean toSync) throws IOException {
ByteArrayOutputStream byteArrayOutputStream = new ByteArrayOutputStream();
final OutputStreamDataOutput out = new OutputStreamDataOutput(new OutputStreamStreamOutput(byteArrayOutputStream));

CodecUtil.writeBEInt(out, CodecUtil.FOOTER_MAGIC);
CodecUtil.writeBEInt(out, 0);
CodecUtil.writeBELong(out, checksum);

Channels.writeToChannel(byteArrayOutputStream.toByteArray(), channel);
if (toSync) {
channel.force(false);
}
}

/**
* validate checks if the Translog file has a valid footer.
* */
static boolean validate(Path path) {
try {
return TranslogFooter.readChecksum(path) != null;
} catch (IOException e) {
return false;
}
}

/**
* readChecksum reads the translog file from the given location and returns the checksum if present in the footer.
* If the translog file is of older version and the footer is not present, then we return null.
* */
static Long readChecksum(Path path) throws IOException {
try (FileChannel channel = FileChannel.open(path, StandardOpenOption.READ)) {
// Read the header and find out if the footer is supported.
final TranslogHeader header = TranslogHeader.read(path, channel);
if (header.getTranslogHeaderVersion() < TranslogHeader.VERSION_WITH_FOOTER) {
return null;

Check warning on line 106 in server/src/main/java/org/opensearch/index/translog/TranslogFooter.java

View check run for this annotation

Codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/TranslogFooter.java#L106

Added line #L106 was not covered by tests
}

// Read the footer.
final long fileSize = channel.size();
final long footerStart = fileSize - TranslogFooter.footerLength();
ByteBuffer footer = ByteBuffer.allocate(TranslogFooter.footerLength());
int bytesRead = Channels.readFromFileChannel(channel, footerStart, footer);
if (bytesRead != TranslogFooter.footerLength()) {
throw new IOException(
"Read " + bytesRead + " bytes from footer instead of expected " + TranslogFooter.footerLength() + " bytes"

Check warning on line 116 in server/src/main/java/org/opensearch/index/translog/TranslogFooter.java

View check run for this annotation

Codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/TranslogFooter.java#L115-L116

Added lines #L115 - L116 were not covered by tests
);
}
footer.flip();

// Validate the footer and return the checksum.
int magic = footer.getInt();
if (magic != CodecUtil.FOOTER_MAGIC) {
throw new IOException("Invalid footer magic number: " + magic);
}

int algorithmId = footer.getInt();
if (algorithmId != 0) {
throw new IOException("Unsupported checksum algorithm ID: " + algorithmId);

Check warning on line 129 in server/src/main/java/org/opensearch/index/translog/TranslogFooter.java

View check run for this annotation

Codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/TranslogFooter.java#L129

Added line #L129 was not covered by tests
}

return footer.getLong();
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -65,11 +65,13 @@ public final class TranslogHeader {
public static final int VERSION_CHECKSUMS = 1; // pre-2.0 - unsupported
public static final int VERSION_CHECKPOINTS = 2; // added checkpoints
public static final int VERSION_PRIMARY_TERM = 3; // added primary term
public static final int CURRENT_VERSION = VERSION_PRIMARY_TERM;
public static final int VERSION_WITH_FOOTER = 4; // added the footer for the translog
public static final int CURRENT_VERSION = VERSION_WITH_FOOTER;

private final String translogUUID;
private final long primaryTerm;
private final int headerSizeInBytes;
private final int translogHeaderVersion;

/**
* Creates a new translog header with the given uuid and primary term.
Expand All @@ -80,14 +82,17 @@ public final class TranslogHeader {
* All operations' terms in this translog file are enforced to be at most this term.
*/
TranslogHeader(String translogUUID, long primaryTerm) {
this(translogUUID, primaryTerm, headerSizeInBytes(translogUUID));
// When we create Header on the fly, we will use the latest current version as that would include the
// checksum as part of the footer.
this(translogUUID, primaryTerm, headerSizeInBytes(translogUUID), CURRENT_VERSION);
assert primaryTerm >= 0 : "Primary term must be non-negative; term [" + primaryTerm + "]";
}

private TranslogHeader(String translogUUID, long primaryTerm, int headerSizeInBytes) {
private TranslogHeader(String translogUUID, long primaryTerm, int headerSizeInBytes, int headerVersion) {
this.translogUUID = translogUUID;
this.primaryTerm = primaryTerm;
this.headerSizeInBytes = headerSizeInBytes;
this.translogHeaderVersion = headerVersion;
}

public String getTranslogUUID() {
Expand All @@ -110,6 +115,13 @@ public int sizeInBytes() {
return headerSizeInBytes;
}

/**
* Returns the version of the translog header.
* */
public int getTranslogHeaderVersion() {
return translogHeaderVersion;
}

static int headerSizeInBytes(String translogUUID) {
return headerSizeInBytes(CURRENT_VERSION, new BytesRef(translogUUID).length);
}
Expand All @@ -127,7 +139,7 @@ private static int headerSizeInBytes(int version, int uuidLength) {
static int readHeaderVersion(final Path path, final FileChannel channel, final StreamInput in) throws IOException {
final int version;
try {
version = CodecUtil.checkHeader(new InputStreamDataInput(in), TRANSLOG_CODEC, VERSION_CHECKSUMS, VERSION_PRIMARY_TERM);
version = CodecUtil.checkHeader(new InputStreamDataInput(in), TRANSLOG_CODEC, VERSION_CHECKSUMS, CURRENT_VERSION);
} catch (CorruptIndexException | IndexFormatTooOldException | IndexFormatTooNewException e) {
tryReportOldVersionError(path, channel);
throw new TranslogCorruptedException(path.toString(), "translog header corrupted", e);
Expand Down Expand Up @@ -183,7 +195,7 @@ public static TranslogHeader read(final Path path, final FileChannel channel) th
in.read(uuid.bytes, uuid.offset, uuid.length);
// Read the primary term
final long primaryTerm;
if (version == VERSION_PRIMARY_TERM) {
if (version >= VERSION_PRIMARY_TERM) {
primaryTerm = in.readLong();
} else {
assert version == VERSION_CHECKPOINTS : "Unknown header version [" + version + "]";
Expand All @@ -202,7 +214,7 @@ public static TranslogHeader read(final Path path, final FileChannel channel) th
+ channel.position()
+ "]";

return new TranslogHeader(uuid.utf8ToString(), primaryTerm, headerSizeInBytes);
return new TranslogHeader(uuid.utf8ToString(), primaryTerm, headerSizeInBytes, version);
} catch (EOFException e) {
throw new TranslogCorruptedException(path.toString(), "translog header truncated", e);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,7 @@ public class TranslogReader extends BaseTranslogReader implements Closeable {
private final Long translogChecksum;
@Nullable
private final Long checkpointChecksum;
private final Boolean hasFooter;

/**
* Create a translog writer against the specified translog file channel.
Expand All @@ -80,14 +81,16 @@ public class TranslogReader extends BaseTranslogReader implements Closeable {
final FileChannel channel,
final Path path,
final TranslogHeader header,
final Long translogChecksum
final Long translogChecksum,
final Boolean hasFooter
) throws IOException {
super(checkpoint.generation, channel, path, header);
this.length = checkpoint.offset;
this.totalOperations = checkpoint.numOps;
this.checkpoint = checkpoint;
this.translogChecksum = translogChecksum;
this.checkpointChecksum = (translogChecksum != null) ? calculateCheckpointChecksum(checkpoint, path) : null;
this.hasFooter = hasFooter != null ? hasFooter : TranslogFooter.validate(path);
}

private static Long calculateCheckpointChecksum(Checkpoint checkpoint, Path path) throws IOException {
Expand Down Expand Up @@ -118,7 +121,7 @@ public Long getCheckpointChecksum() {
public static TranslogReader open(final FileChannel channel, final Path path, final Checkpoint checkpoint, final String translogUUID)
throws IOException {
final TranslogHeader header = TranslogHeader.read(translogUUID, path, channel);
return new TranslogReader(checkpoint, channel, path, header, null);
return new TranslogReader(checkpoint, channel, path, header, null, null);
}

/**
Expand Down Expand Up @@ -146,9 +149,9 @@ TranslogReader closeIntoTrimmedReader(long aboveSeqNo, ChannelFactory channelFac

IOUtils.fsync(checkpointFile.getParent(), true);

newReader = new TranslogReader(newCheckpoint, channel, path, header, translogChecksum);
newReader = new TranslogReader(newCheckpoint, channel, path, header, translogChecksum, hasFooter);
} else {
newReader = new TranslogReader(checkpoint, channel, path, header, translogChecksum);
newReader = new TranslogReader(checkpoint, channel, path, header, translogChecksum, hasFooter);
}
toCloseOnFailure = null;
return newReader;
Expand Down Expand Up @@ -177,6 +180,23 @@ final public Checkpoint getCheckpoint() {
* reads an operation at the given position into the given buffer.
*/
protected void readBytes(ByteBuffer buffer, long position) throws IOException {
if (hasFooter && header.getTranslogHeaderVersion() == TranslogHeader.VERSION_WITH_FOOTER) {
// Ensure that the read request does not overlap with footer.
long translogLengthWithoutFooter = length - TranslogFooter.footerLength();
if (position >= translogLengthWithoutFooter && position < length) {
throw new EOFException(

Check warning on line 187 in server/src/main/java/org/opensearch/index/translog/TranslogReader.java

View check run for this annotation

Codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/TranslogReader.java#L187

Added line #L187 was not covered by tests
"read requested past last ops into footer. pos [" + position + "] end: [" + translogLengthWithoutFooter + "]"
);
}
// If we are trying to read beyond the last Ops, we need to return EOF error.
long lastPositionToRead = position + buffer.limit();
if (lastPositionToRead > translogLengthWithoutFooter) {
throw new EOFException(

Check warning on line 194 in server/src/main/java/org/opensearch/index/translog/TranslogReader.java

View check run for this annotation

Codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/TranslogReader.java#L194

Added line #L194 was not covered by tests
"trying to read past last ops into footer. pos [" + lastPositionToRead + "] end: [" + translogLengthWithoutFooter + "]"
);
}
}

if (position >= length) {
throw new EOFException("read requested past EOF. pos [" + position + "] end: [" + length + "]");
}
Expand Down
Loading

0 comments on commit f17a378

Please sign in to comment.