perf(data): route MailBackfiller.persistBatch through batched updateHeaderContents #322

Closed
opened 2026-07-04 08:11:07 +00:00 by JMR-dev · 0 comments
JMR-dev commented 2026-07-04 08:11:07 +00:00 (Migrated from github.com)

Follow-up to #310/#320. MailBackfiller.persistBatch (:189) has the same per-row forEach { updateHeaderContent } transaction anti-pattern #310 fixed in MailSyncer. The reusable MessageDao.updateHeaderContents(List) @Transaction method is now in place — route backfill through it (one commit vs N). Left out of #310 scope to avoid touching MailBackfillerTest's coVerify { updateHeaderContent }. Backlog — perf, background path.

Follow-up to #310/#320. `MailBackfiller.persistBatch` (:189) has the same per-row `forEach { updateHeaderContent }` transaction anti-pattern #310 fixed in `MailSyncer`. The reusable `MessageDao.updateHeaderContents(List)` @Transaction method is now in place — route backfill through it (one commit vs N). Left out of #310 scope to avoid touching `MailBackfillerTest`'s `coVerify { updateHeaderContent }`. Backlog — perf, background path.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: JMR-dev/LibreMail#322