-
Notifications
You must be signed in to change notification settings - Fork 141
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
Introduce Merkle Patricia Trie #939
Introduce Merkle Patricia Trie #939
Conversation
Codecov Report
@@ Coverage Diff @@
## main #939 +/- ##
==========================================
- Coverage 88.81% 88.55% -0.26%
==========================================
Files 288 315 +27
Lines 26766 27652 +886
==========================================
+ Hits 23771 24486 +715
- Misses 1466 1620 +154
- Partials 1529 1546 +17
|
00fb143
to
c296062
Compare
57ce5ab
to
13176f1
Compare
8d10129
to
ca3b197
Compare
ca3b197
to
9aa6ab1
Compare
|
public void ForkStates<T>(Guid sourceChainId, Guid destinationChainId, Block<T> branchpoint) | ||
where T : IAction, new() | ||
{ | ||
// Do nothing. |
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.
Is it okay that we don't anything here even throwing / logging error?
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.
Actually, ForkStates(…)
is called in BlockChain<T>.Fork(…)
so if it throws exceptions it will always fail. And it is intended.
aae7bcd
to
a1095bf
Compare
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.
Builds seem to fail 😢
e327345
to
b79cbaa
Compare
Co-Authored-By: Seunghun Lee <waydi1@gmail.com>
Co-Authored-By: Seunghun Lee <waydi1@gmail.com>
It is for RocksDB. Co-Authored-By: Seunghun Lee <waydi1@gmail.com>
Co-Authored-By: Seunghun Lee <waydi1@gmail.com>
See https://docs.microsoft.com/en-us/dotnet/csharp/nullable-references Co-Authored-By: Hong Minhee <hong.minhee@planetariumhq.com>
Reorder items Co-Authored-By: Hong Minhee <hong.minhee@planetariumhq.com>
- `BytesEqualityComparer.Equals(…)` Co-Authored-By: Hong Minhee <hong.minhee@planetariumhq.com>
- `HashNode`
- Update XML documents Co-Authored-By: Ko Chanhyuck <lime_bell@naver.com>
- Fix a missing nullable Co-Authored-By: Seunghun Lee <waydi1@gmail.com>
7d98609
04e8700
to
7d98609
Compare
Note: this pull request is based on #950 . It MUST be merged after #950 merged.
Tasks
ITrie.TryGet
,ITrie.Set
IKeyValueStore
for eachIStore
implementationThere may be more updates or the end.