HomePhabricator

Remove work limit in UpdateForDescendants()
76a76321d2f3Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Remove work limit in UpdateForDescendants()

The work limit served to prevent the descendant walking algorithm from doing
too much work by marking the parent transaction as dirty. However to implement
ancestor tracking, it's not possible to similarly mark those descendant
transactions as dirty without having to calculate them to begin with.

This commit removes the work limit altogether. With appropriate
chain limits (-limitdescendantcount) the concern about doing too much
work inside this function should be mitigated.

Details

Provenance
Suhas Daftuar <sdaftuar@gmail.com>Authored on Oct 21 2015, 14:18
schancelPushed on Jan 5 2018, 21:58
schancelPushed on Jan 5 2018, 21:39
schancelPushed on Jan 5 2018, 21:17
Parents
rSTAGING5de2baa138cd: Rename CTxMemPool::remove -> removeRecursive
Branches
Unknown
Tags
Unknown

Event Timeline

Suhas Daftuar <sdaftuar@gmail.com> committed rSTAGING76a76321d2f3: Remove work limit in UpdateForDescendants() (authored by Suhas Daftuar <sdaftuar@gmail.com>).Mar 14 2016, 16:13