-
Notifications
You must be signed in to change notification settings - Fork 143
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
Separate IStore into IStateStore #950
Separate IStore into IStateStore #950
Conversation
a86265e
to
6b8c473
Compare
Codecov Report
@@ Coverage Diff @@
## main #950 +/- ##
==========================================
+ Coverage 88.13% 88.20% +0.06%
==========================================
Files 276 280 +4
Lines 25270 25458 +188
==========================================
+ Hits 22272 22455 +183
- Misses 1537 1540 +3
- Partials 1461 1463 +2
|
c79eeae
to
af1447b
Compare
public interface IStateStore | ||
{ | ||
/// <summary> | ||
/// Sets states mapped as relation <see cref="Block{T}.Hash"/> → states. |
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.
What does “mapped as relation …” mean?
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.
Sadly, I couldn't find well expression for it 😢. Actually I wanted to say like Stores the key-value pair with blockHash as key, states as value.
4bf7ab2
Co-authored-by: Hong Minhee (洪 民憙) <hong.minhee@gmail.com>
4bf7ab2
to
0f0aaa4
Compare
This pull request separates
IStore
's functions to handle state intoIStateStore
.Also, it's a step among steps to apply Merkle Patricia Trie (#939).