Page MenuHomePhabricator

[metachronik] Simplify reconciliations
ClosedPublic

Authored by bytesofman on Sep 5 2025, 23:37.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABCc45f940bc64d: [metachronik] Simplify reconciliations
Summary

The reconciliation framework was added to deal with an API-heavy indexing process. It is overkill to do this with a cron-job on an indexed server, and its complexity makes it tricky to add new indexes.

Remove complex cron-job driven reconciliation and stick to websocket and block-integrity-driven reconciliation.

Test Plan

npm test

tbh ... the test here is seeing if db compute goes down without impacting the charts. there is nothing security or money critical, and metachronik itself is a bit of an experiment. so it is a bit of "land it and see"

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bytesofman published this revision for review.Sep 5 2025, 23:40
bytesofman edited the summary of this revision. (Show Details)
bytesofman edited the test plan for this revision. (Show Details)
Fabien requested changes to this revision.Sep 8 2025, 07:18
Fabien added a subscriber: Fabien.

A doc cleanup is needed, but this is much better than the hacky cron

apps/metachronik/README.md
152 ↗(On Diff #55515)

Don't bury past designs into the doc, it's only making the doc confusing

This revision now requires changes to proceed.Sep 8 2025, 07:18

remove buried refs to old system from readme, rebase

This revision is now accepted and ready to land.Sep 8 2025, 13:43
This revision was automatically updated to reflect the committed changes.