-
Notifications
You must be signed in to change notification settings - Fork 8
pbss implemention #161
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
Open
ryanmorphl2
wants to merge
62
commits into
main
Choose a base branch
from
ryan/pbss
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
pbss implemention #161
Changes from all commits
Commits
Show all changes
62 commits
Select commit
Hold shift + click to select a range
1485d89
feat: pbss implemention
ryanmorphl2 44392b7
WIP
ryanmorphl2 cb759c6
WIP
ryanmorphl2 fade32f
WIP
ryanmorphl2 4814a62
WIP
ryanmorphl2 0a527a0
WIP
ryanmorphl2 01f4ce7
WIP
ryanmorphl2 811ebab
WIP
ryanmorphl2 bcf0b6c
WIP
ryanmorphl2 6c25201
compact key of storage trie node
ryanmorphl2 62f89aa
rawdb is not found err
ryanmorphl2 5bf5bcb
remove unused
ryanmorphl2 bafb670
Copy nodes opt
ryanmorphl2 7a2557b
WIP
ryanmorphl2 5be65e2
WIP
ryanmorphl2 84eaac3
copy nodes opt
ryanmorphl2 0beacac
WIP
ryanmorphl2 feb48bb
buffer: get node opt
ryanmorphl2 a0ed014
rename trie
ryanmorphl2 bba1e75
rm unused
ryanmorphl2 d92fee3
WIP
ryanmorphl2 b1210db
upgrade path zktrie
ryanmorphl2 c72970d
path trie proof
ryanmorphl2 b37cd5f
add flag --state.scheme
ryanmorphl2 ce557df
WIP
ryanmorphl2 ccd7590
WIP
ryanmorphl2 207825a
WIP
ryanmorphl2 bb12ec6
WIP
ryanmorphl2 6e33fa4
WIP
ryanmorphl2 570a566
merge main
ryanmorphl2 b4f0316
upgrage new trie
ryanmorphl2 9109d84
merge main
ryanmorphl2 d554540
statedb: copy with originRoot
ryanmorphl2 49904bf
difflayer: remove unused cache
ryanmorphl2 9c0f993
trie proof: mark todo
ryanmorphl2 6d9401f
fix typo
ryanmorphl2 0f62c7a
offline prune only in hash schema
ryanmorphl2 dea4aa8
merge main
ryanmorphl2 d8f08de
preimages commit
ryanmorphl2 6d086c5
update zktrie version
ryanmorphl2 eb305cb
return state not available
ryanmorphl2 e844e22
fix comment
ryanmorphl2 804cb32
flush with time interval
ryanmorphl2 1e734d8
flush intercal default time
ryanmorphl2 284315c
decrease flush intercal default time
ryanmorphl2 c78b876
set flush intercal default time
ryanmorphl2 8b62f63
add comment
ryanmorphl2 409206c
check scheme before using db
ryanmorphl2 f4cd325
fix log, set interval time
ryanmorphl2 8f1fd9a
fix log
ryanmorphl2 65ef219
merge main
ryanmorphl2 3eb34e6
merge main
ryanmorphl2 eb9937d
modify flush time logic
ryanmorphl2 034e32a
default use async buffer
ryanmorphl2 66e494e
flush with timer
ryanmorphl2 948d2a6
fix commit done
ryanmorphl2 ebf6d61
fix commit done
ryanmorphl2 3e97152
review pr
ryanmorphl2 5dbe405
set max dirtyBufferSize
ryanmorphl2 e6e8be9
review pr
ryanmorphl2 d2ec13b
merge from main
ryanmorphl2 a8a321c
fix: init with genesis path
ryanmorphl2 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
U256.UnmarshalJSON
accepts non-prefixed and malformed hex stringsU256.UnmarshalJSON
currently skips the prefix/length checks implemented for the other numeric helpers (Uint64
,Big
).As a consequence the following inputs are silently accepted, despite being invalid according to the rest of the API:
To keep the semantics consistent and fail early we can reuse
checkNumberText
, handle the"0x"
edge-case and propagate the error throughwrapTypeError
, e.g.:This mirrors
Big.UnmarshalText
, guarantees the0x
prefix, forbids leading zeroes and yields uniform error messages across helpers.