-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
feat(bin, snapshot): SnapshotSegment
, headers segment & refactors
#4979
Conversation
f2f52da
to
7069860
Compare
7069860
to
3afd3bf
Compare
Codecov Report
... and 27 files with indirect coverage changes
Flags with carried forward coverage won't be shown. Click here to find out more.
|
e7aa3a9
to
166fe4f
Compare
166fe4f
to
d91db27
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm,
i don't mind the additional clap feature
db_path, | ||
log_level, | ||
chain.clone(), | ||
*compression, | ||
InclusionFilter::Cuckoo, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe we just replace this with another arg (defaulting to cuckoo
) and add it to the all_combinations
?
This PR mostly refactors existing headers snapshot implementation in CLI using a separate
Segment
trait inreth-snapshot
crate, along with the introduction of enums for describing snapshotting configuration.The CLI behaviour is unchanged: