-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Closed
Labels
C:mempoolComponent: MempoolComponent: MempoolmaintenanceBug/security fixes, correctness, reliability, performance and general QABug/security fixes, correctness, reliability, performance and general QA
Description
- spec: priority mempool specification #9310
- Link to existing ADR when prioritized mempool was introduced. - Performance and stability are at least as good as the mempool in v0.34 (no regression).
- (Backport priority mempool fixes from v0.35.x to v0.34.x. #8962 ) Backported fixes from 0.35 to 0.34 (v0.34.20).
- Benchmark the heap consumption with the backports.
- Prioritized mempool implemented and tested according to the specification.
- Ability to provide priority to transactions in CheckTx responses, which deliver transactions to the application in priority-based order.
- Transactions are still gossiped in FIFO order.
- We have a clear answer to the question as to whether we need the prioritized mempool when ABCI++ lands
- If not we need to remove the
Priority
field fromCheckTx
. This impacts the release of ABCI++.
- If not we need to remove the
Metadata
Metadata
Assignees
Labels
C:mempoolComponent: MempoolComponent: MempoolmaintenanceBug/security fixes, correctness, reliability, performance and general QABug/security fixes, correctness, reliability, performance and general QA
Type
Projects
Status
Done/Merged