Skip to content

Online Indexer: stop retrying an unresolvable partly-built mismatch - #4454

Open
jjezra wants to merge 4 commits into
FoundationDB:mainfrom
jjezra:indexer_no_futile_partly_built_retries
Open

Online Indexer: stop retrying an unresolvable partly-built mismatch#4454
jjezra wants to merge 4 commits into
FoundationDB:mainfrom
jjezra:indexer_no_futile_partly_built_retries

Conversation

@jjezra

@jjezra jjezra commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

The CONTINUE path in OnlineIndexer.indexingCatcher could re-apply an adjustment that was already in effect. An unresolvable "This index was partly built by another method" mismatch was therefore retried identically until reaching max attempts, ending with a misleading "Too many indexing attempts" error.

  • Every adjustment (fallback to a by-records scan, continuation by the previous source index) is now attempted at most once.
  • The mismatch reported to the caller is the first one, so it names the requested indexing method rather than the internal fallback.

Each policy adjustment in the CONTINUE path is now attempted at most once, and the mismatch reported to the caller is the first one - naming the requested indexing method rather than the internal fallback.
@jjezra
jjezra requested a review from ScottDugas August 14, 2026 19:33
@jjezra jjezra added the bug fix Change that fixes a bug label Aug 14, 2026
@jjezra
jjezra marked this pull request as ready for review August 14, 2026 19:33
@jjezra
jjezra marked this pull request as draft August 14, 2026 19:33
@jjezra
jjezra marked this pull request as ready for review August 17, 2026 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug fix Change that fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant