Skip to content

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.

5 participants