Skip to content

snapshots: send highest manifest slot #5932

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

cali-jumptrading
Copy link
Contributor

@cali-jumptrading cali-jumptrading commented Aug 5, 2025

Repair can start before the snapshot is fully loaded if the snapshot tiles send the highest manifest slot seen so far. The highest manifest slot is typically the incremental snapshot slot, but can be the full snapshot slot if incremental snapshots are disabled. The highest manifest slot is guaranteed to be monotonically increasing. It is sent by the snaprd tile when a snapshot peer is selected.

Makes use of #5777 to resolve snapshots for each peer.

@cali-jumptrading cali-jumptrading force-pushed the cali/snapshot-tiles-slot-message branch 7 times, most recently from 186fa5b to 3bdaafc Compare August 7, 2025 14:32
@cali-jumptrading cali-jumptrading changed the title Cali/snapshot tiles slot message wip snapshots: send highest manifest slot Aug 7, 2025
@cali-jumptrading cali-jumptrading force-pushed the cali/snapshot-tiles-slot-message branch from 3bdaafc to 950ec6b Compare August 7, 2025 14:40
@cali-jumptrading cali-jumptrading marked this pull request as ready for review August 7, 2025 14:45
@cali-jumptrading cali-jumptrading force-pushed the cali/snapshot-tiles-slot-message branch 4 times, most recently from aa7352d to 875936e Compare August 7, 2025 17:38
@cali-jumptrading cali-jumptrading force-pushed the cali/snapshot-tiles-slot-message branch from 875936e to 2673ae1 Compare August 8, 2025 20:16
@cali-jumptrading cali-jumptrading force-pushed the cali/snapshot-tiles-slot-message branch from 2673ae1 to 8af7620 Compare August 13, 2025 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant