-
Notifications
You must be signed in to change notification settings - Fork 588
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
[stateless_validation] Remove old state witness distribution mechanism #11179
[stateless_validation] Remove old state witness distribution mechanism #11179
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #11179 +/- ##
==========================================
+ Coverage 71.24% 71.27% +0.02%
==========================================
Files 784 784
Lines 157961 157847 -114
Branches 157961 157847 -114
==========================================
- Hits 112539 112499 -40
+ Misses 40567 40495 -72
+ Partials 4855 4853 -2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Note: Do not merge till we deploy and check partial witness on forknet |
This PR removes a bunch of things that were being used in the old way of distributing chunk state witness. Changes include
ProcessChunkStateWitnessMessage
toChunkStateWitnessMessage
verify_chunk_state_witness_signature
in epoch managerprocess_signed_chunk_state_witness
andpartially_validate_state_witness
in clientChunkStateWitness
network messageSignedEncodedChunkStateWitness
Note: Do not merge this till all nodes in forknet are deployed with the partial encoded state witness change