Page MenuHomePhabricator
Differential D13467

[Chronik] Index txs; add `/tx/:txid` HTTP endpoint
ClosedPublic
Actions

Authored by tobias_ruck on Mar 25 2023, 21:50.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC9514a0915212: [Chronik] Index txs; add `/tx/:txid` HTTP endpoint
Summary

We add transactions to the DB as new blocks are coming in using TxWriter. Until we index the mempool, we set the time_first_seen to 0.

/tx/:txid returns txs by txid from the DB using the QueryTxs struct; not from the mempool yet though, which will be in a future diff.

We also add a few new protobuf messages: Tx, OutPoint, TxInput, TxOutput, BlockMetadata and Network.

Test Plan

ninja check-crates && ninja && ./test/functional/test_runner.py chronik_tx

Diff Detail

Event Timeline

tobias_ruck created this revision.Mar 25 2023, 21:50
Herald added a reviewer: Restricted Project. · View Herald TranscriptMar 25 2023, 21:50
tobias_ruck requested review of this revision.Mar 25 2023, 21:50
Harbormaster completed remote builds in B22704: Diff 38898.Mar 25 2023, 22:04
tobias_ruck added a child revision: D13468: [Chronik] Add `Mempool`, make `/tx/:txid` return mempool txs.Mar 25 2023, 23:21
tobias_ruck updated this revision to Diff 38912.Mar 27 2023, 10:18

Factor out connect_block and disconnect_block, include block hash and num txs in log

Harbormaster completed remote builds in B22716: Diff 38912.Mar 27 2023, 10:35
Fabien requested changes to this revision.Mar 27 2023, 14:35
Fabien added inline comments.
chronik/chronik-indexer/src/query/txs.rs
50–55 ↗(On Diff #38912)

Nit: no need to instanciate a block reader if the tx is not found

chronik/chronik-proto/proto/chronik.proto
48 ↗(On Diff #38912)

What's the use for that one ? Is there any chronik instance that can index several chains ?

test/functional/chronik_tx.py
28 ↗(On Diff #38912)
171 ↗(On Diff #38912)

dito

This revision now requires changes to proceed.Mar 27 2023, 14:35
tobias_ruck updated this revision to Diff 38929.Mar 27 2023, 15:56

Remove 'network' field from proto; small nits

Harbormaster completed remote builds in B22732: Diff 38929.Mar 27 2023, 16:13
Fabien accepted this revision.Mar 27 2023, 16:17
This revision is now accepted and ready to land.Mar 27 2023, 16:17
Closed by commit rABC9514a0915212: [Chronik] Index txs; add `/tx/:txid` HTTP endpoint (authored by tobias_ruck). · Explain WhyMar 27 2023, 17:32
This revision was automatically updated to reflect the committed changes.
tobias_ruck added a commit: rABC9514a0915212: [Chronik] Index txs; add `/tx/:txid` HTTP endpoint.
tobias_ruck mentioned this in D13564: [Chronik] Remove `Network` proto enum.Apr 4 2023, 14:06
tobias_ruck mentioned this in rABC9e7ee00a415d: [Chronik] Remove `Network` proto enum.Apr 4 2023, 15:59

Revision Contents
Changeset List

PathSize
chronik/
chronik-db/
src/
io/
2 lines
chronik-http/
src/
17 lines
chronik-indexer/
3 lines
src/
52 lines
1 line
query/
9 lines
95 lines
chronik-proto/
proto/
74 lines
test/
functional/
171 lines
test_framework/
1 line

Diff 38898

View Options

Cargo.lock

Loading...
View Options

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

Loading...
View Options

chronik/chronik-http/src/server.rs

Loading...
View Options

chronik/chronik-indexer/Cargo.toml

Loading...
View Options

chronik/chronik-indexer/src/indexer.rs

Loading...
View Options

chronik/chronik-indexer/src/lib.rs

Loading...
View Options

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

Loading...
View Options

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

Loading...
View Options

chronik/chronik-proto/proto/chronik.proto

Loading...
View Options

test/functional/chronik_tx.py

Loading...
View Options

test/functional/test_framework/blocktools.py

Loading...