Skip to content

allow pilot to add reindex indexes#1194

Open
bgentry wants to merge 1 commit intomasterfrom
bg/reindexer-pro-extension
Open

allow pilot to add reindex indexes#1194
bgentry wants to merge 1 commit intomasterfrom
bg/reindexer-pro-extension

Conversation

@bgentry
Copy link
Copy Markdown
Contributor

@bgentry bgentry commented Apr 4, 2026

The reindexer operates on a fixed list of indexes. I'm not certain if we have a compelling use case to expose a generalized config for this, but in the meantime, we have pro features which would benefit from periodic re-indexing.

Add a pilot hook that allows for extra reindex targets while preserving the existing schedule and timeout behavior. The reindexer also filters configured targets through IndexesExist so missing optional indexes are skipped cleanly, and tests pin the default and merged target lists.

The reindexer operates on a fixed list of indexes. I'm not certain if we
have a compelling use case to expose a generalized config for this, but
in the meantime, we have pro features which would benefit from periodic
re-indexing.

Add a pilot hook that allows for extra reindex targets while preserving
the existing schedule and timeout behavior. The reindexer also filters
configured targets through `IndexesExist` so missing optional indexes are
skipped cleanly, and tests pin the default, merged, and deduplicated
target lists.
@bgentry bgentry requested a review from brandur April 4, 2026 00:45
Copy link
Copy Markdown
Contributor

@brandur brandur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool.

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.

2 participants