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

feat(dot/sync): Implement state sync strategy #4425

Draft
wants to merge 13 commits into
base: diego/warpsync/strategy
Choose a base branch
from
Draft

feat(dot/sync): Implement state sync strategy #4425

wants to merge 13 commits into from

Conversation

dimartiro
Copy link
Contributor

@dimartiro dimartiro commented Dec 19, 2024 •
edited
Loading

Changes

Implements state sync strategy as the next step to be able to use the warp sync process.
This change includes:

  • State sync strategy implementation
  • Refactor StateRequestProvider
    • Move it as part of sync package
    • Re use it in state retrieval script
    • Tests
  • Strategy switching
  • Refactor sync service to create the strategies within it as we discussed here
  • Tests

Tests

go test -tags integration github.com/ChainSafe/gossamer

Issues

Closes: #4238

@dimartiro dimartiro linked an issue Dec 19, 2024 that may be closed by this pull request
Implement state sync strategy #4238
Open
@dimartiro dimartiro force-pushed the diego/warpsync/strategy branch from a1b63c3 to 36b57e0 Compare December 19, 2024 16:37
@dimartiro dimartiro force-pushed the diego/statesync/strategy branch from 61d4a98 to 956d49d Compare December 19, 2024 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@EclesioMeloJunior EclesioMeloJunior Awaiting requested review from EclesioMeloJunior EclesioMeloJunior will be requested when the pull request is marked ready for review EclesioMeloJunior is a code owner

@jimjbrettj jimjbrettj Awaiting requested review from jimjbrettj jimjbrettj will be requested when the pull request is marked ready for review jimjbrettj is a code owner

@timwu20 timwu20 Awaiting requested review from timwu20 timwu20 will be requested when the pull request is marked ready for review timwu20 is a code owner

@P1sar P1sar Awaiting requested review from P1sar P1sar will be requested when the pull request is marked ready for review P1sar is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Implement state sync strategy
1 participant
@dimartiro