Page MenuHomePhabricator

[avalanche] Enable mining finalized transactions
ClosedPublic

Authored by Fabien on Jul 23 2025, 13:09.

Details

Reviewers
PiRK
Group Reviewers
Restricted Project
Commits
rABCff646f2281f2: [avalanche] Enable mining finalized transactions
Summary

This require to only include the transactions that are still in the mempool.
Because the node keeps the radix tree untouched until the block is finalized, it would be possible for the node to generate a block template with confirmed transactions otherwise.

The feature is gated behind a dedicated flag for now. In the end this will be tied to preconsensus but this requires some tests to be updated first.

Test Plan
ninja check-all

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Fabien published this revision for review.Jul 23 2025, 14:00
PiRK added inline comments.
test/functional/abc_mining_avalanche_preconsensus.py
139–140 ↗(On Diff #54979)
147 ↗(On Diff #54979)

looks good, except for the comment nits

This revision is now accepted and ready to land.Jul 24 2025, 07:58