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

feat(dot/state): Introduce BlockState & StorageState interfaces #4528

Merged
merged 20 commits into from
Feb 12, 2025

Conversation

dimartiro
Copy link
Contributor

@dimartiro dimartiro commented Feb 10, 2025

Changes

Introduce BlockState and StorageState interface to decouple the usage from the current implementation.

  • Introduce BlockState and StorageState interfaces
  • Rename current BlockState implementation to DefaultBlockState
  • Remove old BlockState and StorageState interfaces other packages and centralize everything in state.BlockState and state.StorageState
  • Fix tests

Tests

make test

Issues

Close: #4459

@timwu20 timwu20 requested a review from P1sar as a code owner February 11, 2025 13:53
@dimartiro dimartiro merged commit 45f3f51 into refactor/client-db Feb 12, 2025
24 checks passed
@dimartiro dimartiro deleted the diego/state_interfaces branch February 12, 2025 16:55
timwu20 added a commit that referenced this pull request Feb 13, 2025
Co-authored-by: Timothy Wu <tim.wu@chainsafe.io>
Co-authored-by: Haiko Schol <hs@haikoschol.com>
timwu20 added a commit that referenced this pull request Feb 13, 2025
Co-authored-by: Timothy Wu <tim.wu@chainsafe.io>
Co-authored-by: Haiko Schol <hs@haikoschol.com>
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

Successfully merging this pull request may close these issues.

2 participants