Differential D15455

[lib] Limit the batch size
ClosedPublic
Actions

Authored by tomek on Oct 13 2025, 4:06 PM.

Details

Reviewers

kamil

Commits

rCOMM9caa243ba217: [lib] Limit the batch size

Summary

If a batch exceeds the limit, move messages to the next batch.

https://linear.app/comm/issue/ENG-11360/reduce-the-size-of-batches-when-syncing-farcaster-data

Test Plan

Set a really low limit and check if the sync works correctly.

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

tomek created this revision.Oct 13 2025, 4:06 PM

Herald added a subscriber: ashoat. · View Herald TranscriptOct 13 2025, 4:06 PM

Harbormaster returned this revision to the author for changes because remote builds failed.Oct 13 2025, 4:18 PM

Harbormaster failed remote builds in B35922: Diff 50620!

Harbormaster completed remote builds in B35922: Diff 50620.Oct 14 2025, 8:56 AM

tomek requested review of this revision.Oct 14 2025, 8:56 AM

kamil accepted this revision.Oct 14 2025, 12:03 PM

This revision is now accepted and ready to land.Oct 14 2025, 12:03 PM

Closed by commit rCOMM9caa243ba217: [lib] Limit the batch size. · Explain WhyOct 14 2025, 12:54 PM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMM9caa243ba217: [lib] Limit the batch size.

Revision Contents
Changeset List

Path

Size

lib/

shared/

farcaster/

farcaster-hooks.js

83 lines

Diff 50620

View Options

lib/shared/farcaster/farcaster-hooks.js

[lib] Limit the batch sizeClosedPublicActions