Fix: Preserve order of searchable attributes when modified #5751
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Bug Description
When modifying the
searchableAttributes
setting through the settings API, the order of attributes was not being preserved. The system would incorrectly detect that no changes had occurred when only the order was modified, causing the update to be skipped.Example:
searchableAttributes: ["title", "overview"]
searchableAttributes: ["overview", "title"]
Root Cause
The bug was in the change detection logic within
update_user_defined_searchable_attributes()
incrates/milli/src/update/settings.rs
. The method was comparing the new user-defined order against the computedsearchable_fields
instead of theuser_defined_searchable_fields
.Since
searchable_fields
returns computed fields that may have a different order than the user-defined ones, order changes were not being detected.Fix
Changed the comparison to use
user_defined_searchable_fields
instead:This ensures that any change in the user-defined order is properly detected and applied.