Page MenuHomePhabricator

[backport#15159#15404] [RPC] Remove lookup to UTXO set from GetTransaction
AbandonedPublic

Authored by majcosta on May 13 2020, 17:37.

Details

Reviewers
jasonbcox
Group Reviewers
Restricted Project
Summary

[RPC] Update getrawtransaction interface https://github.com/bitcoin/bitcoin/pull/15159/commits/04da9f4834e1651da65ceb6379950cef9450591c (Amiti Uttarwar)

Address test todos by removing -txindex to nodes. https://github.com/bitcoin/bitcoin/pull/15404/commits/8e4b4f683a0b342cec24cd51b1e98433034ea2ea (Amiti Uttarwar)


Depends on D6058

This is a backport of Core PR15159 and PR15404 due to PR15159 breaking rpc_psbt.py functional test

Test Plan
ninja check-all

Event Timeline

majcosta created this revision.May 13 2020, 17:37
Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 13 2020, 17:37
majcosta requested review of this revision.May 13 2020, 17:37
teamcity edited the summary of this revision. (Show Details)May 13 2020, 17:37

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

jasonbcox requested changes to this revision.May 13 2020, 18:13
jasonbcox added a subscriber: jasonbcox.

The diff title doesn't match the change. It looks like we're missing a backport or something? Check out https://github.com/bitcoin/bitcoin/pull/15159

This revision now requires changes to proceed.May 13 2020, 18:13
majcosta abandoned this revision.May 13 2020, 20:44
majcosta retitled this revision from [backport#15404] Address test todos by removing -txindex to nodes. Originally added when updating getrawtransaction to stop searching unspent utxos. to [backport#15159#15404] [RPC] Remove lookup to UTXO set from GetTransaction.May 13 2020, 21:38
majcosta edited the summary of this revision. (Show Details)
majcosta planned changes to this revision.May 13 2020, 21:39
majcosta abandoned this revision.May 13 2020, 22:36