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.

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

Get chain syncing to spec #3567

Closed
ZenGround0 opened this issue Oct 16, 2019 · 1 comment
Closed

Get chain syncing to spec #3567

ZenGround0 opened this issue Oct 16, 2019 · 1 comment

Comments

@ZenGround0
Copy link
Contributor

Description

This is a stub issue tracking updates in the design of the filecoin chain syncing subsystem in light of recent updates to the spec. I'm writing a design doc here and spiking code changes at #3565

Acceptance criteria

We can rapidly implement the spec requirements for chain syncing.

Risks + pitfalls

Lots of new details coming from the spec require us to quickly make significant changes. This design change is going to be complex. The syncing spec is about 60% there. There are a lot of unspecified details, particularly in the CHAIN_CATCHUP phase. There are also serious disagreements about key parts of the protocol (what does block pubsub message contain is on top of the list). This design is attempting to see the forest through the trees and set us up for success regardless of where we come out on this issues.

Where to begin

If you haven't already, catch up on the spec changes: filecoin-project/specs#568. Read the design doc once I ping you here. Then take a look at the WIP spike PR. Both places are good for comments.

@anorth
Copy link
Member

anorth commented Mar 1, 2020

@ZenGround0 done?

@anorth anorth closed this as completed Mar 2, 2020
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

No branches or pull requests

2 participants