Skip to content

mempool: Removed the ancestor_score index

Calin Culianu requested to merge cculianu/bitcoin-cash-node:mempool_nuke_ancestor_score into master

Summary

This MR depends on !1078 (merged), !1118 (merged) and !1080 (merged).

This commit removes the index for ancestor_score and the comparator CompareTxMemPoolEntryByAncestorFee, since after !1078 (merged) and !1080 (merged) are merged it will no longer be needed.

This index was previously used for CPFP mining, which we no longer do after !1078 (merged).

Test Plan

Nothing is using this index now after !1078 (merged) , other than unit test code to test this index, which has been removed.

  • ninja check check-functional

Note to reviewers

You should just review the top-most commit. The other commits are just the commits from !1078 (merged), !1080 (merged), and !1118 (merged) !

Edited by Calin Culianu

Merge request reports