Paths

Table of Contentst

Differential D13759

[Chronik] Index spent_by + add `spent_by` to `Tx` protobuf
ClosedPublic
Actions

Authored by tobias_ruck on Apr 23 2023, 13:18.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABCe08f7ce5dd29: [Chronik] Index spent_by + add `spent_by` to `Tx` protobuf

Summary

Add which outputs have been spent by which input using SpentByWriter for the DB and MempoolSpentBy for the mempool.

Add the spent_by field to TxOutput in the protobuf; this uses the OutPoint message, note that this points to an input.

Also add a OutputsSpent helper to query us the required data to tell which outputs have been spent by which tx.

Depends on D13768.

Test Plan

ninja && ninja ./test/functional/test_runner.py chronik_spent_by

Diff Detail

Repository

rABC Bitcoin ABC

Branch

chronik-spent-by

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 23402
Build 46423: Build Diff	build-debug · build-without-wallet · build-clang-tidy · build-diff · build-clang · build-chronik
Build 46422: arc lint + arc unit

Event Timeline

tobias_ruck created this revision.Apr 23 2023, 13:18

Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 23 2023, 13:18

tobias_ruck requested review of this revision.Apr 23 2023, 13:18

Harbormaster completed remote builds in B23402: Diff 39865.Apr 23 2023, 13:33

tobias_ruck added a child revision: D13760: [Chronik] Add `MempoolGroupUtxos` to store mempool UTXOs by group.Apr 23 2023, 20:11

Use SpentBy

Harbormaster completed remote builds in B23417: Diff 39885.Apr 24 2023, 14:21

Rebase onto chronik-refactor-query-txs

tobias_ruck edited the summary of this revision. (Show Details)Apr 24 2023, 14:43

tobias_ruck added a parent revision: D13768: [Chronik] Refactor `QueryTxs::tx_by_id`: remove Cow etc..

Harbormaster completed remote builds in B23420: Diff 39891.Apr 24 2023, 15:16

Fabien accepted this revision.Apr 24 2023, 16:28

This revision is now accepted and ready to land.Apr 24 2023, 16:28

Closed by commit rABCe08f7ce5dd29: [Chronik] Index spent_by + add `spent_by` to `Tx` protobuf (authored by tobias_ruck). · Explain WhyApr 24 2023, 19:45

This revision was automatically updated to reflect the committed changes.

tobias_ruck added a commit: rABCe08f7ce5dd29: [Chronik] Index spent_by + add `spent_by` to `Tx` protobuf.

Revision Contents
Changeset List

Path

Size

chronik/

chronik-db/

src/

db.rs

3 lines

io/

spent_by.rs

5 lines

mem/

mempool.rs

21 lines

chronik-indexer/

src/

indexer.rs

11 lines

query/

group_history.rs

21 lines

txs.rs

88 lines

util.rs

114 lines

chronik-proto/

proto/

chronik.proto

5 lines

test/

functional/

chronik_spent_by.py

168 lines

Diff 39865

View Options

chronik/chronik-db/src/db.rs

View Options

chronik/chronik-db/src/io/spent_by.rs

View Options

chronik/chronik-db/src/mem/mempool.rs

View Options

chronik/chronik-indexer/src/indexer.rs

View Options

chronik/chronik-indexer/src/query/group_history.rs

View Options

chronik/chronik-indexer/src/query/txs.rs

View Options

chronik/chronik-indexer/src/query/util.rs

View Options

chronik/chronik-proto/proto/chronik.proto

View Options

test/functional/chronik_spent_by.py

[Chronik] Index spent_by + add `spent_by` to `Tx` protobufClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 39865

chronik/chronik-db/src/db.rs

chronik/chronik-db/src/io/spent_by.rs

chronik/chronik-db/src/mem/mempool.rs

chronik/chronik-indexer/src/indexer.rs

chronik/chronik-indexer/src/query/group_history.rs

chronik/chronik-indexer/src/query/txs.rs

chronik/chronik-indexer/src/query/util.rs

chronik/chronik-proto/proto/chronik.proto

test/functional/chronik_spent_by.py

[Chronik] Index spent_by + add `spent_by` to `Tx` protobuf
ClosedPublic
Actions

Revision Contents
Changeset List