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

Get whoosh backend working properly with fuzzy search. #1534

Open
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@december1981

december1981 commented Jul 19, 2017

See comments on related commits.

@december1981

This comment has been minimized.

Show comment
Hide comment
@december1981

december1981 Jul 19, 2017

Owner

~%d based on what I understand from here:
http://whoosh.readthedocs.io/en/latest/parsing.html

Note, not sure how to deal with "HAYSTACK_FUZZY_MAX_EXPANSIONS" - it may be that whoosh doesn't support defining that constraint explicitly. (I also note mutations greater than 2 can be very slow).

Owner

december1981 commented on ea9ed0a Jul 19, 2017

~%d based on what I understand from here:
http://whoosh.readthedocs.io/en/latest/parsing.html

Note, not sure how to deal with "HAYSTACK_FUZZY_MAX_EXPANSIONS" - it may be that whoosh doesn't support defining that constraint explicitly. (I also note mutations greater than 2 can be very slow).

@acdha

This comment has been minimized.

Show comment
Hide comment
@acdha

acdha Jul 19, 2017

Contributor

HAYSTACK_FUZZY_MAX_EXPANSIONS is only supported by the ES backend currently so perhaps we should make that an option specific to that backend if nothing else supports it.

Contributor

acdha commented Jul 19, 2017

HAYSTACK_FUZZY_MAX_EXPANSIONS is only supported by the ES backend currently so perhaps we should make that an option specific to that backend if nothing else supports it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment