Page MenuHomePhabricator

[Chronik] Add `ScriptGroup` to group txs by script
ClosedPublic

Authored by tobias_ruck on Apr 3 2023, 17:09.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC6bd5d170cf6e: [Chronik] Add `ScriptGroup` to group txs by script
Summary

An implementation of Group for grouping by script.

We cannot use ffi::compress_script directly here, as the tests can't link against that, so instead it is a field in ScriptGroup.

We also remove the chronik-bridge dependency as we don't need it.

Test Plan

ninja check-crates

Diff Detail

Repository
rABC Bitcoin ABC
Branch
chronik-script-group
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 22944
Build 45508: Build Diffbuild-chronik
Build 45507: arc lint + arc unit

Event Timeline

tobias_ruck created this revision.
Fabien requested changes to this revision.Apr 4 2023, 08:17
Fabien added inline comments.
chronik/chronik-db/Cargo.toml
16 ↗(On Diff #39171)

That doesn't belong to this diff ?

This revision now requires changes to proceed.Apr 4 2023, 08:17

split out removal of chronik-bridge dependency

This revision is now accepted and ready to land.Apr 4 2023, 10:01