Skip to content

mempool: fix getrawmempool priority on bridge nodes - #444

Merged
kcalvinalvin merged 1 commit into
utreexo:mainfrom
kcalvinalvin:2026-10-06-fix-rawmempoolverbose-priority
Oct 8, 2026
Merged

kcalvinalvin merged 1 commit into
utreexo:mainfrom
kcalvinalvin:2026-10-06-fix-rawmempoolverbose-priority

Conversation

@kcalvinalvin

@kcalvinalvin kcalvinalvin commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

On bridge nodes, and any node run with --noutreexo, getrawmempool with verbose set reported a currentpriority of 0 for every transaction.

RawMempoolVerbose looked up inputs in the utxo set only when IsUtreexoViewActive was nil and calling it also returned false. The server always sets IsUtreexoViewActive, so the condition was never true and these nodes looked up inputs in poolLeaves instead. poolLeaves has no entries on these nodes, so no inputs were found.

Look up inputs in the utxo set when IsUtreexoViewActive is nil or returns false. CSNs take the same path as before.

Validation: go test ./mempool, golangci-lint run

@kcalvinalvin
kcalvinalvin force-pushed the 2026-10-06-fix-rawmempoolverbose-priority branch from 16244e7 to 55961ef Compare October 7, 2026 08:03
On bridge nodes, and any node run with --noutreexo, getrawmempool with
verbose set reported a currentpriority of 0 for every transaction.

RawMempoolVerbose looked up inputs in the utxo set only when
IsUtreexoViewActive was nil and calling it also returned false. The
server always sets IsUtreexoViewActive, so the condition was never
true and these nodes looked up inputs in poolLeaves instead.
poolLeaves has no entries on these nodes, so no inputs were found.

Look up inputs in the utxo set when IsUtreexoViewActive is nil or
returns false.
@kcalvinalvin
kcalvinalvin force-pushed the 2026-10-06-fix-rawmempoolverbose-priority branch from 55961ef to 2fc8f59 Compare October 7, 2026 08:22
@kcalvinalvin kcalvinalvin changed the title mempool: use the utxo set for priority without a utreexo view mempool: fix getrawmempool priority on bridge nodes Oct 7, 2026
@kcalvinalvin
kcalvinalvin merged commit 8083a79 into utreexo:main Oct 8, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant