Skip to content

Commit

Permalink
[Kernel] Further refactor and cleanup SnapshotManager (#4090)
Browse files Browse the repository at this point in the history
#### Which Delta project/connector is this regarding?

- [ ] Spark
- [ ] Standalone
- [ ] Flink
- [X] Kernel
- [ ] Other (fill in here)

## Description

This PR further refactors and cleans up SnapshotManager. We consolidate
all of our `getLogSegment` methods to just one. It has a deep
implementation. This PR adds nice, clean header blocks to describe the
steps during log segment construction.

## How was this patch tested?

Refactor. Update existing UTs.

## Does this PR introduce _any_ user-facing changes?

No.
  • Loading branch information
scottsand-db authored Jan 29, 2025
1 parent 0c58836 commit c04cb06
Show file tree
Hide file tree
Showing 4 changed files with 292 additions and 403 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ public static KernelException versionBeforeFirstAvailableCommit(
return new KernelException(message);
}

public static KernelException versionAfterLatestCommit(
public static KernelException versionToLoadAfterLatestCommit(
String tablePath, long versionToLoad, long latestVersion) {
String message =
String.format(
Expand Down
Loading

0 comments on commit c04cb06

Please sign in to comment.