Skip to content

feat: prioritized mempool#480

Merged
tzdybal merged 4 commits intomainfrom
tzdybal/prioritized_mempool
Aug 1, 2022
Merged

feat: prioritized mempool#480
tzdybal merged 4 commits intomainfrom
tzdybal/prioritized_mempool

Conversation

@tzdybal
Copy link
Copy Markdown
Contributor

@tzdybal tzdybal commented Jul 30, 2022

This PR brings prioritized mempool from Tendermint v0.34.20 to Optimint.

mempool was copied from tendermint v0.34.20
It was adapted to make it compatible:

  • tmsync package was removed (sync used instead)
  • clist copied to mempool/, imports fixed
  • v0 removed
  • v1/reactor.go removed

Changes in node, rpc, state packages was caused by changes in mempool
API.

Resolves: #191

tzdybal added 2 commits July 30, 2022 22:09
mempool was copied from tendermint v0.34.20
It was adapted to make it compatible:
 - tmsync package was removed (sync used instead)
 - clist copied to mempool/, imports fixed
 - v0 removed
 - v1/reactor.go removed

Changes in node, rpc, state packages was caused by changes in mempool
API.
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 30, 2022

Codecov Report

Merging #480 (0e407a9) into main (30fa397) will increase coverage by 0.99%.
The diff coverage is 80.34%.

@@            Coverage Diff             @@
##             main     #480      +/-   ##
==========================================
+ Coverage   55.15%   56.15%   +0.99%     
==========================================
  Files          48       50       +2     
  Lines        8802     9237     +435     
==========================================
+ Hits         4855     5187     +332     
- Misses       3193     3292      +99     
- Partials      754      758       +4     
Impacted Files Coverage Δ
mempool/metrics.go 15.51% <10.52%> (-2.44%) ⬇️
rpc/client/client.go 45.94% <50.00%> (ø)
mempool/mempool.go 52.77% <53.33%> (-29.84%) ⬇️
mempool/clist/clist.go 75.00% <75.00%> (ø)
mempool/v1/tx.go 76.31% <76.31%> (ø)
mempool/cache.go 84.21% <84.21%> (ø)
mempool/v1/mempool.go 86.72% <86.72%> (ø)
node/node.go 65.32% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us.

@tzdybal tzdybal enabled auto-merge July 30, 2022 21:48
Comment thread mempool/ids_test.go Outdated
Comment thread mempool/ids.go Outdated
Comment thread mempool/mempool.go Outdated
@tzdybal tzdybal requested a review from liamsi July 31, 2022 21:38
Copy link
Copy Markdown
Contributor

@liamsi liamsi left a comment

Choose a reason for hiding this comment

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

utACK

@tzdybal tzdybal merged commit 705f608 into main Aug 1, 2022
@tzdybal tzdybal deleted the tzdybal/prioritized_mempool branch August 1, 2022 05:41
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.

Update mempool

3 participants