Streamline EndBlock #2540
Streamline EndBlock #2540
Conversation
|
The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).
|
|
The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).
|
Codecov Report❌ Patch coverage is ❌ Your patch check has failed because the patch coverage (16.23%) is below the target coverage (70.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #2540 +/- ##
==========================================
- Coverage 43.37% 43.32% -0.05%
==========================================
Files 1576 1578 +2
Lines 137816 137767 -49
==========================================
- Hits 59775 59694 -81
- Misses 72622 72654 +32
Partials 5419 5419
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
There was a problem hiding this comment.
CodeQL found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
codchen
force-pushed
the
tony/end-block
branch
from
0ca8b94 to
f0dd230
Compare
| k.RemoveFirstNTxHashes(ctx, DefaultTxHashesToRemove) | ||
|
|
||
| // Migrate legacy EVM receipts to receipt.db in small batches every N blocks | ||
| if ctx.BlockHeight()%LegacyReceiptMigrationInterval == 0 { |
There was a problem hiding this comment.
I think is not needed any more, @jewei1997 can you confirm?
Describe your changes and provide context
Flatten call stack of EndBlock
Testing performed to validate your change
unit test