Skip to content

Conversation

@AliZDev-v0
Copy link

@AliZDev-v0 AliZDev-v0 commented Jan 9, 2026

PR description

There is one big PR which resolve the issues #9561
Since initial PR contains 200+ files and heavy for review it will be separated onto 4-5 smaller PR's

  • User plugin.data.BlockHeader in ethereum/core
  • Move WorldStatePreimageStorage, WorldStateKeyValueStorage,
  • Create interface for config classes WorldStateQueryParams, WorldStateConfig
  • Move MutableWorldState, StateRootCommitter
  • Create WorldStateArchiveProvider and minor changes if any left

Fixed Issue(s)

#9448

Thanks for sending a pull request! Have you done the following?

  • Checked out our contribution guidelines?
  • Considered documentation and added the doc-change-required label to this PR if updates are required.
  • Considered the changelog and included an update if required.
  • For database changes (e.g. KeyValueSegmentIdentifier) considered compatibility and performed forwards and backwards compatibility tests

Locally, you can run these tests to catch failures early:

  • spotless: ./gradlew spotlessApply
  • unit tests: ./gradlew build
  • acceptance tests: ./gradlew acceptanceTest
  • integration tests: ./gradlew integrationTest
  • reference tests: ./gradlew ethereum:referenceTests:referenceTests
    EIPBlockchainReferenceTest_281.java is failing, but it fails on my local master as well
  • hive tests: Engine or other RPCs modified?

Signed-off-by: Ali Zhagparov <alijakparov.kz@gmail.com>
Signed-off-by: Ali Zhagparov <alijakparov.kz@gmail.com>
@AliZDev-v0 AliZDev-v0 force-pushed the ali/use-block-header-plugin branch from 06d184a to f34589c Compare January 10, 2026 06:13
Signed-off-by: Ali Zhagparov <alijakparov.kz@gmail.com>
Signed-off-by: Ali Zhagparov <alijakparov.kz@gmail.com>
@AliZDev-v0 AliZDev-v0 marked this pull request as ready for review January 12, 2026 00:34
@AliZDev-v0
Copy link
Author

@jflo can you review this PR, as discussed breaking PR into smaller for review
Initial PR: #9561

@macfarla macfarla moved this to Open PRs in 26.1.0 Release Jan 22, 2026
@AliZDev-v0
Copy link
Author

AliZDev-v0 commented Jan 22, 2026

Thanks for paying attention to my PR 😃
I will resolve conflicts soon

Signed-off-by: Ali Zhagparov <alijakparov.kz@gmail.com>
Signed-off-by: Ali Zhagparov <alijakparov.kz@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Open PRs

Development

Successfully merging this pull request may close these issues.

2 participants