Skip to content
This repository has been archived by the owner on Jun 9, 2024. It is now read-only.

refactor(txpool): Use Geth TxPool #1041

Closed
wants to merge 48 commits into from
Closed

Conversation

itsdevbear
Copy link
Member

No description provided.

@linear
Copy link

linear bot commented Aug 29, 2023

PLR-107 Geth TxPool

use wrapped geth txpool as app-side mempool, but not changing the block building yet. Refer to https://github.com/berachain/polaris/pull/810/files

@codecov
Copy link

codecov bot commented Aug 29, 2023

Codecov Report

Merging #1041 (bf268c8) into main (0c53dfe) will decrease coverage by 2.20%.
The diff coverage is 23.15%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1041      +/-   ##
==========================================
- Coverage   54.11%   51.92%   -2.20%     
==========================================
  Files         101      100       -1     
  Lines        5353     5242     -111     
==========================================
- Hits         2897     2722     -175     
- Misses       2282     2345      +63     
- Partials      174      175       +1     
Files Changed Coverage Δ
cosmos/x/evm/keeper/genesis.go 0.00% <ø> (ø)
cosmos/x/evm/plugins/block/plugin.go 31.81% <0.00%> (-1.52%) ⬇️
cosmos/x/evm/plugins/configuration/plugin.go 27.27% <ø> (+2.27%) ⬆️
cosmos/x/evm/plugins/gas/plugin.go 73.80% <ø> (ø)
cosmos/x/evm/plugins/txpool/handler.go 0.00% <0.00%> (ø)
cosmos/x/evm/plugins/txpool/plugin.go 0.00% <0.00%> (ø)
cosmos/x/evm/plugins/txpool/serializer.go 0.00% <0.00%> (ø)
e2e/testapp/app.go 1.58% <0.00%> (-0.01%) ⬇️
eth/core/chain.go 0.00% <0.00%> (ø)
eth/core/chain_reader.go 0.00% <ø> (ø)
... and 8 more

@itsdevbear itsdevbear changed the base branch from main to reeee-fuck August 30, 2023 04:23
@mergify mergify bot mentioned this pull request Aug 30, 2023
eth/polar/backend.go Outdated Show resolved Hide resolved
ocnc
ocnc previously approved these changes Sep 11, 2023
Copy link
Contributor

@ocnc ocnc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, it survived the load test

@itsdevbear
Copy link
Member Author

closing in favour of #1140

@itsdevbear itsdevbear closed this Sep 26, 2023
@calbera calbera deleted the calbera/plr-107-geth-txpool branch November 8, 2023 18:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants