Skip to content

cherry-pick: Index backfills resume from their checkpoint and yield the event loop, so large tables converge (harper#2536) (conflicts → v5.2) - #2551

Merged
kriszyp merged 2 commits into
v5.2from
cherry-pick/v5.2/pr-2539
Sep 10, 2026
Merged

kriszyp merged 2 commits into
v5.2from
cherry-pick/v5.2/pr-2539

Conversation

@github-actions

@github-actions github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Cherry-pick of PR #2539 onto `v5.2` produced conflicts on commit(s): `2e8171fa95598c2c9b6507176d80d8a364533e88`.

Resolve the conflict markers on branch `cherry-pick/v5.2/pr-2539` and merge this PR.

@claude please review branch `cherry-pick/v5.2/pr-2539` and suggest a patch that resolves the conflict markers (<<<<<<< / ======= / >>>>>>>) introduced by cherry-picking PR #2539 onto `v5.2`. Post the suggested patch as a comment here — do not push.

Index backfills resume from their checkpoint and yield the event loop, so large tables converge (harper#2536)
…test's descriptor lookup

The cherry-pick of PR #2539 onto v5.2 left conflict markers around
runIndexing's new preamble: v5.2's signature takes no branchPath (branches
are main-only), so the resolution keeps the three-parameter signature and
every added helper (checkpoint rate limiting, flushIndexStores,
resumeStartKey, per-put rejection tracking). The resulting diff against v5.2
is line-for-line identical to PR #2539's diff on main.

indexRestartNumber.test.js's findDescriptor scanned the whole dbisDB, which
the 'test' database shares across suites, so the new
indexBackfillConvergence tables' own `tag` attribute was returned instead
and two assertions failed. Scoped the scan to the table's key prefix,
matching the fix already on main (#2258).

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@kriszyp
kriszyp merged commit 7dfa82f into v5.2 Sep 10, 2026
@kriszyp
kriszyp deleted the cherry-pick/v5.2/pr-2539 branch September 10, 2026 01:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant