Page MenuHomePhabricator
Differential D15547

[Chronik] `ReverseLookup`: Remove redundant `lookup_cf_name`
ClosedPublic
Actions

Authored by tobias_ruck on Feb 26 2024, 09:16.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABCcfbddd174fec: [Chronik] `ReverseLookup`: Remove redundant `lookup_cf_name`
Summary

This has always been in the generic param L already, so we can just use that.

Test Plan

ninja check-crates && ninja check-functional

Diff Detail

Repository
rABC Bitcoin ABC
Branch
chronik-reverse_lookup-remove-arg
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 27412
Build 54387: Build Diffchronik-client-integration-tests · build-chronik · build-chronik-plugins
Build 54386: arc lint + arc unit

Event Timeline

tobias_ruck created this revision.Feb 26 2024, 09:16
Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 26 2024, 09:16
tobias_ruck requested review of this revision.Feb 26 2024, 09:16
Harbormaster completed remote builds in B27412: Diff 45597.Feb 26 2024, 09:52
teamcity added a comment.Feb 26 2024, 09:52
The build failed due to an unexpected infrastructure outage. The administrators have been notified to investigate. Sorry for the inconvenience.
tobias_ruck added a child revision: D15548: [Chronik] Use `merge_cf` in `reverse_lookup.rs` to speed up indexing..Feb 26 2024, 10:45
Fabien accepted this revision.Feb 26 2024, 14:27
This revision is now accepted and ready to land.Feb 26 2024, 14:27
tobias_ruck added a comment.Feb 26 2024, 15:16

CI failure unrelated

This revision was landed with ongoing or failed builds.Feb 26 2024, 15:17
Closed by commit rABCcfbddd174fec: [Chronik] `ReverseLookup`: Remove redundant `lookup_cf_name` (authored by tobias_ruck). · Explain Why
This revision was automatically updated to reflect the committed changes.
tobias_ruck added a commit: rABCcfbddd174fec: [Chronik] `ReverseLookup`: Remove redundant `lookup_cf_name`.

Revision Contents
Changeset List

PathSize
chronik/
chronik-db/
src/
io/
7 lines
2 lines
15 lines

Diff 45597

View Options

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

Loading...
View Options

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

Loading...
View Options

chronik/chronik-db/src/reverse_lookup.rs

Loading...