Skip to content

MB-66112: fix an edge case related to index convergence#2200

Merged
Thejas-bhat merged 1 commit into
masterfrom
convergeBug
May 28, 2025
Merged

MB-66112: fix an edge case related to index convergence#2200
Thejas-bhat merged 1 commit into
masterfrom
convergeBug

Conversation

@Thejas-bhat

Copy link
Copy Markdown
Member
  • As an edge case, the segment layout can be such that the merge planning generates tasks having only a single non-empty segment.
  • When this task is processed and introduced into the scorch system, the root doesn't change and you'd be stuck in an infinite loop because the same segment is still present in the system and the plan keeps generating the task with the same single segment, which doesn't converge the index to a steady state.

@Thejas-bhat Thejas-bhat merged commit e8d4425 into master May 28, 2025
9 checks passed
@CascadingRadium CascadingRadium deleted the convergeBug branch May 29, 2025 03:25
CascadingRadium pushed a commit that referenced this pull request Aug 21, 2025
- As an edge case, the segment layout can be such that the merge
planning generates tasks having only a single non-empty segment.
- When this task is processed and introduced into the scorch system, the
root doesn't change and you'd be stuck in an infinite loop because the
same segment is still present in the system and the plan keeps
generating the task with the same single segment, which doesn't converge
the index to a steady state.
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.

3 participants