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

[WIP] all: enable pbss #600

Draft
wants to merge 12 commits into
base: path-base-implementing
Choose a base branch
from

Conversation

Francesco4203
Copy link
Collaborator

@Francesco4203 Francesco4203 commented Oct 10, 2024

Enabling PBSS for ready to test. Including fixed commits for making sure the PBSS can run well with unittest and local test.

@Francesco4203 Francesco4203 force-pushed the enable-pbss branch 5 times, most recently from 89ec36f to 027e92a Compare October 10, 2024 11:36
@huyngopt1994 huyngopt1994 force-pushed the path-base-implementing-starting branch from 85dc7e2 to 2cbb47a Compare October 10, 2024 15:11
@huyngopt1994 huyngopt1994 changed the base branch from path-base-implementing-starting to path-base-implementing October 11, 2024 04:19
@huyngopt1994 huyngopt1994 changed the title all: enable pbss [WIP] all: enable pbss Oct 11, 2024
@Francesco4203 Francesco4203 force-pushed the enable-pbss branch 7 times, most recently from b7aa791 to 7b2746d Compare October 14, 2024 09:26
Francesco4203 and others added 9 commits October 15, 2024 13:30
- all: fix and enable tests for pathbase
- blockchain: open triedb explicitly in blockchain functions and close right after use, since diskLayer inside pathdb is a skeleton
- blockchain: when writeBlockWithState, pathbase will skip the explicit garbage collector, which is only needed for hashbase
- genesis.go: nit: change check genesis state, ref ethereum/go-ethereum@08bf8a6
- all: fix tests, enable path scheme tests
- state_accessor: split function to retrieve statedb from block to hash scheme and path scheme
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