-
Notifications
You must be signed in to change notification settings - Fork 25.1k
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
Deprecate min_score
#13115
Comments
Actually it is probably not a low hanging fruit because of eg. #14300. |
@jpountz looks like this is still relevant, but please correct me if I'm wrong |
It is still relevant indeed, and also #14300 is no longer an issue. I removed the |
I am ++ on deprecating min score. For function scores it's fine, but for other queries it's a bad idea as it will work in a way that cuts results but in advance it's not possible to know what score is suitable to be used, and score can vary. |
Pinging @elastic/es-search (Team:Search) |
Pinging @elastic/es-search-relevance (Team:Search Relevance) |
Quoting documentation: "Note, most times, this does not make much sense, but is provided for advanced use cases".
Besides, it can be implemented easily with a function_score query, which will be as efficient and more flexible since you can put it anywhere in the query tree.
The text was updated successfully, but these errors were encountered: