-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
allow exporting a number of recent chain state trees #3463
Conversation
wait like, 20 minutes, observe creation of 10gb file.
Skip entire sync process! |
cmd/lotus/daemon.go
Outdated
snapshot := cctx.String("checkpoint") | ||
if chainfile != "" || snapshot != "" { | ||
if chainfile != "" && snapshot != "" { | ||
return fmt.Errorf("cannot specify both 'snapshot' and 'import-chain'") |
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.
return fmt.Errorf("cannot specify both 'snapshot' and 'import-chain'") | |
return fmt.Errorf("cannot specify both 'checkpoint' and 'import-chain'") |
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.
checkpoint
is a term that carries social consensus connotations.- i think snapshot is the right term for the exported state. (ie not all snapshots one may load should be thought of as checkpoints)
- maybe the daemon load flag should be
lotus daemon --load-state=snapshot.car
orlotus daemon --load-snapshot=snapshot.car
Thanks so much -- this will help so many people trying to sync the chain, specially from scratch 🙏 |
0dd75a1
to
2c1a640
Compare
568bcb1
to
59f765f
Compare
Gets us closer to being able to 'sync' faster.