-
Notifications
You must be signed in to change notification settings - Fork 67
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
Matches all docs if bf (Boost Function) present #65
Comments
Yeah, I've been having this problem as well with Solr 6.0 and the 5.0.4 version of the plugin; I wasn't sure what was causing it. Thanks for working on this @janhoy ! |
Would you have a chance to test my PR and see if it works for you? If so, I'll commit it.. |
Absolutely. I'll check it out tomorrow morning and give it a test run. |
Everything seems to be working correctly. I haven't come across any query yet that throws it off. I think it may be good to go! |
Thanks @janhoy for the work you put into this. I really appreciate it! 😺 |
If your edismax query involves
bf
parameter, and the query matches a synonym, this plugin seems to generate a query which always returns all documents. Simply adding e.g.bf=last_modified
to any of the unit tests will reveal the bug. The constructed parsed-query looks something like this (assuming query restskatt being expanded with synonym baksmell, and bf=SearchRank^10:The text was updated successfully, but these errors were encountered: