Parses, colors, validates and indexes mempool txs as SLP/ALP txs.
In fetch_tx_spent_tokens, bulk-query ops are used wherever possible. It is exposed so we can use it in the indexer later, too.
Details
Details
- Reviewers
Fabien - Group Reviewers
Restricted Project - Commits
- rABC9fe11d18aec3: [Chronik] Add `MempoolTokens` to index mempool token txs
cargo test -p chronik-db test_mempool_tokens
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Branch
- chronik-slp-mempool
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 26184 Build 51940: Build Diff chronik-client-integration-tests · build-chronik Build 51939: arc lint + arc unit
Event Timeline
| chronik/chronik-db/src/mem/tokens.rs | ||
|---|---|---|
| 38 | Please add the word token somewhere in this message, otherwise it can be understood as if the mempool was corrupted | |
| 89 | Since this is expected to not fail as the genesis should be in a block already, should you have some error in the else case to give feedback to the user? | |
| 164 | What happens if the tx is in the mempool but the vout doesn't exist ? | |
| 195 | Please comment: "Not a token spendable output" | |
| 777 | ||
| 804 | Can you remove txid(10) and check the input is not found on txid(11) instead ? You get more coverage | |
Comment Actions
mention "token" in error messages; add explanatory comments; fix typo; test tx 10 too
| chronik/chronik-db/src/mem/tokens.rs | ||
|---|---|---|
| 804 | Not sure why exactly; the mempool expects txs to be removed in reverse-topological order and 11 depends on 10, so it'll just sperg | |
Comment Actions
The build failed due to an unexpected infrastructure outage. The administrators have been notified to investigate. Sorry for the inconvenience.
Comment Actions
The build failed due to an unexpected infrastructure outage. The administrators have been notified to investigate. Sorry for the inconvenience.