Skip to content
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

[EVM] Add transient receipts with eventual flush to store #1742

Merged
merged 27 commits into from
Jun 27, 2024

Conversation

stevenlanders
Copy link
Contributor

@stevenlanders stevenlanders commented Jun 21, 2024

Describe your changes and provide context

  • receipts are written to a transient store
  • transient store is flushed via baseapp's commit to a state store
  • this will help prevent state bloat

Testing performed to validate your change

  • unit tests
  • integration tests
  • load tests

app/app.go Fixed Show fixed Hide fixed
app/test_state_store.go Dismissed Show dismissed Hide dismissed
app/test_state_store.go Dismissed Show dismissed Hide dismissed
app/test_state_store.go Fixed Show fixed Hide fixed
app/test_state_store.go Fixed Show fixed Hide fixed
app/test_state_store.go Dismissed Show dismissed Hide dismissed
app/test_state_store.go Dismissed Show dismissed Hide dismissed
app/test_state_store.go Dismissed Show dismissed Hide dismissed
app/test_state_store.go Dismissed Show dismissed Hide dismissed
Copy link

codecov bot commented Jun 26, 2024

Codecov Report

Attention: Patch coverage is 68.00000% with 96 lines in your changes missing coverage. Please review.

Project coverage is 60.91%. Comparing base (6e9257d) to head (7876601).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1742      +/-   ##
==========================================
+ Coverage   60.88%   60.91%   +0.02%     
==========================================
  Files         372      373       +1     
  Lines       26832    27116     +284     
==========================================
+ Hits        16337    16517     +180     
- Misses       9403     9488      +85     
- Partials     1092     1111      +19     
Files Coverage Δ
x/evm/keeper/keeper.go 49.73% <100.00%> (+0.27%) ⬆️
x/evm/keeper/msg_server.go 76.17% <100.00%> (ø)
x/evm/module.go 55.88% <100.00%> (ø)
x/evm/types/keys.go 23.25% <ø> (ø)
app/test_helpers.go 60.16% <92.30%> (+1.87%) ⬆️
app/app.go 66.23% <70.00%> (+<0.01%) ⬆️
x/evm/keeper/receipt.go 60.71% <66.66%> (-7.71%) ⬇️
app/test_state_store.go 65.85% <65.85%> (ø)

... and 2 files with indirect coverage changes

app/test_state_store.go Dismissed Show dismissed Hide dismissed
@stevenlanders stevenlanders changed the title [WIP] Add transient receipts with eventual flush to store [EVM] Add transient receipts with eventual flush to store Jun 27, 2024
@stevenlanders stevenlanders merged commit d25456d into main Jun 27, 2024
51 checks passed
@stevenlanders stevenlanders deleted the add-transient-receipts branch June 27, 2024 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants