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

Changes necessary for Stateless Witness Building #29029

Closed
wants to merge 7 commits into from
Closed

Changes necessary for Stateless Witness Building #29029

wants to merge 7 commits into from

Conversation

jwasinger
Copy link
Contributor

@jwasinger jwasinger commented Feb 20, 2024 •
edited
Loading

This implements:

  • The necessary changes to construct a stateless block witness.
  • A debug API method debug_buildStatelessProof to construct a witness for a given block.
  • A blockchain test runner which constructs a witness for each successfully-imported block.

This PR pulls changes from #28776, omitting:

  • stateless execution implementation, and the standalone utility/sub-command which exposes a stateless block execution endpoint behind an HTTP server.
  • stateless cross-validation logic in core/blockchain.go during block import.

@jwasinger
add necessary changes for building stateless block witnesses. Expose …
9100e2e 
…a debug api method to build witness for a given block. Add blockchain test suite which builds witnesses for each imported block. Stateless execution changes and cross-validation logic omitted to minimize the code footprint of this PR
@jwasinger
fix
095a65e
@jwasinger
other fix
93557ab
@jwasinger
more
d7ba47e
@jwasinger
more
100d2e3
@jwasinger
more
d6d48f3
@jwasinger
more
3a121ee
@jwasinger jwasinger marked this pull request as ready for review February 20, 2024 05:17
@jwasinger
Copy link
Contributor Author

I still need to fix the unit tests broken by this.

@jwasinger jwasinger mentioned this pull request Feb 20, 2024
Closed
@jwasinger jwasinger mentioned this pull request Mar 8, 2024
Merged
@jwasinger jwasinger mentioned this pull request Apr 15, 2024
Merged
@jwasinger jwasinger closed this Apr 30, 2024
@jwasinger jwasinger reopened this Apr 30, 2024
@jwasinger jwasinger closed this Apr 30, 2024
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@karalabe karalabe Awaiting requested review from karalabe karalabe is a code owner

@holiman holiman Awaiting requested review from holiman holiman is a code owner

@rjl493456442 rjl493456442 Awaiting requested review from rjl493456442 rjl493456442 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.
2 participants
@jwasinger @lightclient