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.

Sign up for GitHub

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

Jump to bottom

fatxpool: proper handling of priorities when mempool is full #6647

Draft
wants to merge 39 commits into
base: master
Choose a base branch
from
Draft

fatxpool: proper handling of priorities when mempool is full #6647

wants to merge 39 commits into from

Conversation

michalkucharczyk
Copy link
Contributor

Higher priority transaction replaces lower priority transaction also when internal mempool is full.

closes: #5809

michalkucharczyk and others added 30 commits November 12, 2024 15:29
@michalkucharczyk
flaky test fixed
e0a7bc6
@michalkucharczyk
dropped_watcher: explicit view removal
354d6a3
@michalkucharczyk
fatp: prios some intial tests added
1151556
@michalkucharczyk
base_pool: handling conflicts in future improved
c9f2d39
@michalkucharczyk
dropped_watcher: rename C -> ChainApi
406c37b
@michalkucharczyk
base_pool: handling conflicts in future (2)
fb5e355
@michalkucharczyk
fatp: DroppedTransaction + DroppedReason added
e798e34
@michalkucharczyk
graph: dedicated usurped/limit_enforced calls added
7b58a68
@michalkucharczyk
dropped_watcher: todo added
322774c
@michalkucharczyk
test: prios tests
a116c20
@michalkucharczyk
dropped_watcher: dropping future transcations improved
560db28
@michalkucharczyk
test fixed
5a83cf5
@michalkucharczyk
tests: future dropping test added
c566dad
@michalkucharczyk
fatp: TimedTransactionSource added
f10590f
@michalkucharczyk
adjusting tests
7b96ca3
@michalkucharczyk
fixes
69567b8
@michalkucharczyk
fatp: better support for usurped transactions
414ec3c
@actions-user
Update from michalkucharczyk running command 'prdoc --bump major --au…
0cf6043 
…dience node_dev'
@michalkucharczyk
one more test + future_at added
44a0096
@michalkucharczyk
clippy
d69c660
@michalkucharczyk
is_imported: fix
b47b174
@michalkucharczyk
fatp: avoid some duplications in update_view_with_mempool
2d0bbf8
@michalkucharczyk
prdoc updated
31b3392
@michalkucharczyk
misspell
f66b1ad
@michalkucharczyk
prodc
4449969
@michalkucharczyk
Merge remote-tracking branch 'origin/master' into mku-fatxpool-priori…
4855881 
…ties
@michalkucharczyk
api: async_trait
26d27ec
@michalkucharczyk
api: async_trait - rpc
e27e9b5
@michalkucharczyk
Merge remote-tracking branch 'origin/master' into mku-fatxpool-mempoo…
2a6190a 
…l-priorities-at-limits
@michalkucharczyk
dropped_watcher: new fns are public
b413f2b
@michalkucharczyk michalkucharczyk force-pushed the mku-fatxpool-mempool-priorities-at-limits branch from 724c102 to 4338671 Compare November 26, 2024 12:52
@paritytech-workflow-stopper
Copy link

All GitHub workflows were cancelled due to failure one of the required jobs.
Failed workflow url: https://github.com/paritytech/polkadot-sdk/actions/runs/12073389837
Failed job name: cargo-clippy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

fatxpool: cross check (and improve) support for priority transactions
2 participants
@michalkucharczyk @actions-user