Skip to content
This repository has been archived by the owner on Apr 4, 2023. It is now read-only.

Accept integers as document ids again #617

Merged
merged 1 commit into from
Aug 31, 2022
Merged

Conversation

Kerollmops
Copy link
Member

This PR is related to meilisearch/meilisearch#2723 and will fix when this PR will be merged, a new release deployed and used in Meilisearch itself.

This PR makes the indexer to try to parse the values of the fields identified as numbers i.e. id:number as integer first then as float if it fails.

@Kerollmops Kerollmops added the no breaking The related changes are not breaking (DB nor API) label Aug 31, 2022
@Kerollmops Kerollmops requested a review from irevoire August 31, 2022 09:00
@Kerollmops Kerollmops changed the title Accept again integers as document ids Accept integers as document ids again Aug 31, 2022
Copy link
Member

@irevoire irevoire left a comment

Choose a reason for hiding this comment

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

Thank you for the fix & test!
bors merge

@bors
Copy link
Contributor

bors bot commented Aug 31, 2022

@bors bors bot merged commit 2c2f3d3 into main Aug 31, 2022
@bors bors bot deleted the fix-integer-as-document-id branch August 31, 2022 09:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
no breaking The related changes are not breaking (DB nor API)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants