Skip to content

Reindex orchestrator workflow adjustments#5406

Open
SergeyGaluzo wants to merge 48 commits intomainfrom
users/sergal/reindex-remove-explicit-counts-2
Open

Reindex orchestrator workflow adjustments#5406
SergeyGaluzo wants to merge 48 commits intomainfrom
users/sergal/reindex-remove-explicit-counts-2

Conversation

@SergeyGaluzo
Copy link
Copy Markdown
Contributor

@SergeyGaluzo SergeyGaluzo commented Feb 23, 2026

This PR is the prerequisite for removing explicit counts from reindex.
Scope of changes:

  1. Simplifies logic by removing "early exists" based on zero counts and empty lists when possible.
  2. Introduces "transient" dictionaries to facilitate reindex completion management.
  3. Simplifies logic to derive final statuses.
  4. Eliminates redundant calls to the database to get search param statuses.
  5. Improves scalability of getting job status calls.

@SergeyGaluzo SergeyGaluzo requested a review from a team as a code owner February 23, 2026 01:00
@SergeyGaluzo SergeyGaluzo changed the title [do not review] remove explicit counts in reindex Reindex orchestrator refactoring Mar 2, 2026
@SergeyGaluzo SergeyGaluzo added the Enhancement Enhancement on existing functionality. label Mar 2, 2026
@SergeyGaluzo SergeyGaluzo added this to the CY25Q3/2Wk07 milestone Mar 2, 2026
@SergeyGaluzo SergeyGaluzo changed the title Reindex orchestrator refactoring Reindex orchestrator workflow adjustments Mar 4, 2026
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 29, 2026

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement Enhancement on existing functionality.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants