Skip to content

snapshot: Provide streaming snapshot verification. #2691

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
May 8, 2025
Merged

Conversation

kim
Copy link
Contributor

@kim kim commented May 2, 2025

Repurposes the remote SnapshotFetcher for verification of the snapshot's objects. This can be useful when memory usage is of concern.

@kim kim requested a review from Shubham8287 May 2, 2025 15:05
@CLAassistant
Copy link

CLAassistant commented May 3, 2025

CLA assistant check
All committers have signed the CLA.

@bfops bfops added the release-any To be landed in any release window label May 5, 2025
Copy link
Contributor

@Shubham8287 Shubham8287 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@kim kim added this pull request to the merge queue May 8, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 8, 2025
@kim kim added this pull request to the merge queue May 8, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 8, 2025
kim added 3 commits May 8, 2025 17:02
Repurposes the remote `SnapshotFetcher` for verification of the
snapshot's objects. This can be useful when memory usage is of concern.
@kim kim force-pushed the kim/snapshot/verify branch from 4cdfc4e to e7533bf Compare May 8, 2025 15:02
@kim kim enabled auto-merge May 8, 2025 15:09
auto-merge was automatically disabled May 8, 2025 15:17

Pull Request is not mergeable

@kim kim enabled auto-merge May 8, 2025 15:25
@kim kim added this pull request to the merge queue May 8, 2025
Merged via the queue into master with commit e32fc4a May 8, 2025
29 of 30 checks passed
@kim kim deleted the kim/snapshot/verify branch May 8, 2025 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-any To be landed in any release window
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants