Freegle / Iznik / 27958

73%

Build Type

push

circleci

Committed by invalid-email-address

Commit Message

fix(matched): slice the backfill so it doesn't blow up the DB connection

The first --backfill drove buildNotifications on all ~24k open posts at once:
its long apiv2 phase idled the DB connection out and loadOpenMessages then hit
'2006 MySQL server has gone away' on a giant WHERE id IN(...). Walk the open-age
window in bounded time slices instead (backfill_slice_minutes, default 360):
freshPosts now takes an optional upper bound and buildNotifications an explicit
(since, until), so each slice drives on a small, bounded set — short apiv2
phase, small queries, and each slice's sends persist before the next (partial
progress survives a mid-run failure). Extracted dispatchNotifications() so the
scheduled run and each backfill slice share one delivery path.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Coverage Stats

13216 of 17415 branches covered (75.89%)

Branch coverage included in aggregate %.

55 of 76 new or added lines in 2 files covered. (72.37%)

10 existing lines in 4 files now uncovered.

139014 of 192578 relevant lines covered (72.19%)

37.91 hits per line