Search with all words #29

Open
wants to merge 2 commits into
from

Projects

None yet

3 participants

@buzztaiki

I implement new option that allow to search with all words.
It works as well on my trac :)

@t2y
Contributor
t2y commented Apr 7, 2014

Looks good to me.

@t2y
Contributor
t2y commented Apr 11, 2014

ping?

@dnephin
Owner
dnephin commented Apr 15, 2014

Hello, this seems to be a pretty major change to how searches are constructed. Why is this necessary?

I will need to do some more manual testing and I don't have the time for that today. I'll look into this further.

@buzztaiki

Thank you for your reply.

this seems to be a pretty major change to how searches are constructed. Why is this necessary?

Because, edismax parser ignores mm and q.op parameter when use explicit operator (see this issue).

I changed query construction for avoiding this issue using nested query.

I will need to do some more manual testing and I don't have the time for that today. I'll look into this further.

Sorry...

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