The event log's resubmit count reads every live event instead of using the resubmitted_from_id index #325

New Issue

2026-09-28T13:36:41+02:00

clawbot commented

2026-09-28 13:36:41 +02:00

resubmitCounts in internal/handlers/source_management.go runs on every event log page: it counts how many events were resubmitted from each of the page's events, with resubmitted_from_id IN (...) over up to 25 event ids. GORM adds deleted_at IS NULL, and SQLite, which keeps no statistics on the per-webhook databases, then reads through the deleted_at index instead of idx_events_resubmitted_from_id whenever the page holds two or more events. Every page load reads every live event in that webhook's database.

This is the cause #319 fixes for the other event-tier indexes, by adding deleted_at to each index.

Definition of done

The statement resubmitCounts sends for a page of events uses an index on resubmitted_from_id, shown by a test that checks SQLite's plan for that statement as GORM builds it, as #319 does for its statements.
An existing per-webhook database gains the index on open.
The README's Event-tier indexes table lists it.

Model: opus-5-5

`resubmitCounts` in `internal/handlers/source_management.go` runs on every event log page: it counts how many events were resubmitted from each of the page's events, with `resubmitted_from_id IN (...)` over up to 25 event ids. GORM adds `deleted_at IS NULL`, and SQLite, which keeps no statistics on the per-webhook databases, then reads through the `deleted_at` index instead of `idx_events_resubmitted_from_id` whenever the page holds two or more events. Every page load reads every live event in that webhook's database. This is the cause https://git.eeqj.de/sneak/webhooker/pulls/319 fixes for the other event-tier indexes, by adding `deleted_at` to each index. ## Definition of done - The statement `resubmitCounts` sends for a page of events uses an index on `resubmitted_from_id`, shown by a test that checks SQLite's plan for that statement as GORM builds it, as https://git.eeqj.de/sneak/webhooker/pulls/319 does for its statements. - An existing per-webhook database gains the index on open. - The README's Event-tier indexes table lists it. Model: opus-5-5

clawbot referenced this issue

2026-09-28 13:37:17 +02:00

Index the event-tier columns the sweeps, event log and retention scan #319

clawbot referenced this issue

2026-09-28 13:37:28 +02:00

Index the event-tier columns the sweeps, event log and retention scan #319

clawbot referenced this issue from a commit

2026-09-28 14:13:24 +02:00

Index the event-tier columns the sweeps, event log and retention scan (closes #314)

Sign in to join this conversation.