Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stem prefix items #30553

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Stem prefix items #30553

wants to merge 1 commit into from

Conversation

bratseth
Copy link
Member

@bratseth bratseth commented Mar 9, 2024

If we are searching a stemmed index, it's probably better to stem terms also when we are searching for prefixes.

If we are searching a stemmed index, it's probably better to
stem terms also when we are searching for prefixes.
@bratseth bratseth requested a review from arnej27959 March 9, 2024 20:24
@bratseth
Copy link
Member Author

To quickly see the point of this scroll down to the bottom of the changes.

Copy link
Member

@arnej27959 arnej27959 left a comment

Choose a reason for hiding this comment

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

I'm still not sure if this is helpful. Searching in a (stemmed or not) index with a prefix item is probably a mistake, since indexes does not currently support prefix search. Maybe it should be flagged with an error message by backend in that case?

@bratseth
Copy link
Member Author

Yes, you're right. I want to add support for prefix search in indexes though, and maybe also stemmed attributes, but we can let this wait for that.

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