-
Notifications
You must be signed in to change notification settings - Fork 709
[MEL] - Implement L2 messages accumulation and introduce MessageReader to extract messages from preimages #4258
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
base: master
Are you sure you want to change the base?
Conversation
…ract messages from preimages
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #4258 +/- ##
==========================================
- Coverage 32.94% 29.27% -3.67%
==========================================
Files 471 474 +3
Lines 56610 56688 +78
==========================================
- Hits 18648 16597 -2051
- Misses 34713 37094 +2381
+ Partials 3249 2997 -252 |
❌ 3 Tests Failed:
View the top 3 failed tests by shortest run time
📣 Thoughts on this report? Let Codecov know! | Powered by Codecov |
rauljordan
left a comment
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.
This looks really good to me so far, no need to keep in draft IMO
The base branch was changed.
Realized that some issues will happen with the current code because we changed delayed msg hashes
| crypto.Keccak256(m.Message.L2msg), | ||
| ) | ||
| return crypto.Keccak256Hash(hash) | ||
| encoded, err := rlp.EncodeToBytes(m.WithMELRelevantFields()) |
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.
@ganeshvanahalli won't this break existing code? We want to eventually merge to master and cannot break existing logic
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.
Hash() method is only used by MEL code currently, the rest of the codebase (tracker, reader etc..) uses AfterInboxAcc method
This PR implements accumulating of L2 messages in every mel state by introducing two new fields-
We also introduce
MessageReaderstruct in melreplay package that will be use by the unified replay binary to read L2 messages from preimagesResolves NIT-3522