Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Change snap-sync UX for early history. #3044

Merged
merged 4 commits into from
Sep 23, 2024
Merged

Change snap-sync UX for early history. #3044

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
e832f28
Panic on insufficient archived history during snap-sync.
shamil-gadelshin Sep 20, 2024
73ec690
Make full snap sync as a default mode.
shamil-gadelshin Sep 20, 2024
6b69b93
Revert "Make full snap sync as a default mode."
shamil-gadelshin Sep 23, 2024
a47f45a
Fix string formatting.
shamil-gadelshin Sep 23, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions crates/subspace-service/src/sync_from_dsn/snap_sync.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -167,12 +167,12 @@ where
}
};

// Skip the snap sync if there is just one segment header built on top of genesis, it is
// more efficient to sync it regularly
// We don't have the genesis state when we choose to snap sync.
if target_segment_index <= SegmentIndex::ONE {
debug!("Snap sync was skipped due to too early chain history");

return Ok(None);
panic!(
"Snap sync is impossible - not enough archived history: \
wipe the DB folder and rerun with --sync=full"
);
}

// Identify all segment headers that would need to be reconstructed in order to get first
Expand Down
Loading