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

Hotfix for search performance (meilisearch#1707) #378

Merged
merged 3 commits into from
Sep 29, 2021
Merged

Conversation

ManyTheFish
Copy link
Member

This PR contains an ugly quick fix of meilisearch#1707.

  • remove comparison reverse on rank. Enhancing relevancy and performances
  • iterate over level 0 only. Enhancing performances.

A better fix is in development.

milli/src/search/criteria/attribute.rs Outdated Show resolved Hide resolved
milli/src/search/criteria/attribute.rs Outdated Show resolved Hide resolved
ManyTheFish and others added 2 commits September 29, 2021 14:49
Co-authored-by: Clément Renault <clement@meilisearch.com>
Co-authored-by: Clément Renault <clement@meilisearch.com>
Copy link
Member

@Kerollmops Kerollmops left a comment

Choose a reason for hiding this comment

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

Nice, merging this hotfix!
bors merge

@bors
Copy link
Contributor

bors bot commented Sep 29, 2021

@bors bors bot merged commit 6a057a3 into main Sep 29, 2021
@bors bors bot deleted the hotfix-meilisearch-1707 branch September 29, 2021 13:20
@curquiza curquiza changed the title Hotfix meilisearch#1707 Hotfix for search performance (meilisearch#1707) Sep 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants