-
Notifications
You must be signed in to change notification settings - Fork 136
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
docs: 13690 Added a design doc for Ledger State API
#13730
Conversation
Node: HAPI Test (Crypto) Results 24 files 24 suites 13m 27s ⏱️ Results for commit 34efbae. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Node Death Reconnect) Results3 tests 3 ✅ 6m 17s ⏱️ Results for commit 34efbae. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Misc) Results 51 files 51 suites 21m 40s ⏱️ Results for commit 34efbae. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Smart Contract) Results 73 files 73 suites 23m 52s ⏱️ Results for commit 34efbae. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Time Consuming) Results19 tests 19 ✅ 23m 28s ⏱️ Results for commit 34efbae. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Restart) Results3 tests 3 ✅ 7m 10s ⏱️ Results for commit 34efbae. ♻️ This comment has been updated with latest results. |
Node: Unit Test Results 1 540 files 1 540 suites 2h 42m 17s ⏱️ Results for commit 34efbae. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Token) Results 20 files 20 suites 5m 59s ⏱️ Results for commit 34efbae. ♻️ This comment has been updated with latest results. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #13730 +/- ##
=============================================
- Coverage 67.58% 58.27% -9.31%
+ Complexity 36486 22080 -14406
=============================================
Files 3732 2883 -849
Lines 149767 109662 -40105
Branches 15571 11159 -4412
=============================================
- Hits 101223 63910 -37313
+ Misses 44179 42219 -1960
+ Partials 4365 3533 -832 ☔ View full report in Codecov by Sentry. |
Refactor for State Proofs
Refactor for State Proofs
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.
+1
platform-sdk/docs/proposals/001-merkle-state-api/001-merkle-state-api.md
Outdated
Show resolved
Hide resolved
platform-sdk/docs/proposals/001-merkle-state-api/001-merkle-state-api.md
Outdated
Show resolved
Hide resolved
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.
+1
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.
+1
Public Merkle State API
Ledger State API
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.
+1
platform-sdk/docs/proposals/001-ledger-state-api/001-ledger-state-api.md
Outdated
Show resolved
Hide resolved
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.
+1
platform-sdk/docs/proposals/001-ledger-state-api/001-ledger-state-api.md
Show resolved
Hide resolved
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.
+1
Signed-off-by: Austin Littley <austin@swirldslabs.com>
e4cf729
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.
+1
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.
+0.99 - Do these changes need to be reflected in HIP anywhere?
+0.9 the proposal is great but I do not have enough knowledge of the technical topic to do +1 |
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.
+1 the hip I asked about would come through block-streams at the very latest.
Description:
This design doc is a compilation of the following Notion pages:
State Refactor for State Proofs
Public Specified Merkle Tree & State
Public Merkle Tree API
Merkle tree to protobuf
Proposed Platform State Structure
Fixes #13690
Notes for reviewer:
Checklist