Skip to content

Keep the search engine configured on an index when deploying index definitions - #5950

Merged
ramonsmits merged 1 commit into
release-6.21from
backport-5946-release-6.21
Oct 5, 2026
Merged

ramonsmits merged 1 commit into
release-6.21from
backport-5946-release-6.21

Conversation

@ramonsmits

@ramonsmits ramonsmits commented Oct 5, 2026 •

Copy link
Copy Markdown
Member

Backport to release-6.21 of:

Fixes:

Cherry-pick of 53e287d, no conflicts. The IndexSetupTests pass on this branch: Audit 13/13, Primary 1/1.

See #5946 for the problem, the fix and the verification of the upgrade paths.

…finitions (#5946)

* Keep the search engine configured on an index when deploying index definitions

Operators migrating indexes from Corax to Lucene in RavenDB Studio store the
search engine in the index configuration. ServiceControl deployed its index
definitions without it, so RavenDB built a side-by-side replacement that fell
back to the database default, which is Corax for databases created before
6.20. Unless the index was locked, every start-up reset migrated indexes back
to Corax.

Index definitions are now deployed carrying over the search engine configured
on the existing index (or on its pending replacement). A migrated index no
longer needs to be locked, still receives definition changes, and a pending
Corax replacement created by earlier versions is discarded at start-up.

----

* Create new indexes with Lucene and respect search engines set in code

Indexes that don't exist on the server yet are now created with the Lucene
search engine pinned on the index, also in existing databases that still
default to Corax. A search engine set explicitly in an index definition,
through SearchEngineType or Configuration, takes precedence over the one on
the server. Expand the comments to explain why the search engine has to be
resolved before deploying the index definitions.
---------

Co-authored-by: Ramon Smits <ramon.smits@gmail.com>
(cherry picked from commit 53e287d)
@ramonsmits
ramonsmits marked this pull request as ready for review October 5, 2026 14:35
@ramonsmits
ramonsmits merged commit 09eeadd into release-6.21 Oct 5, 2026
36 checks passed
@ramonsmits
ramonsmits deleted the backport-5946-release-6.21 branch October 5, 2026 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants