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

MLT returns all documents if non of the fields in the document are supported #3453

Closed
s1monw opened this issue Aug 7, 2013 · 0 comments
Closed

Comments

@s1monw
Copy link
Contributor

s1monw commented Aug 7, 2013

Today due the optimizations in the boolean query builder we adjust
a pure negative query with a 'match_all'. This is not the desired
behavior in the MLT API if all the fields in a document are unsupported ie. numeric fields etc. If that happens today we return all documents but the one MLT is
executed on.

@ghost ghost assigned s1monw Aug 7, 2013
s1monw added a commit to s1monw/elasticsearch that referenced this issue Aug 7, 2013
Today due the optimizations in the boolean query builder we adjust
a pure negative query with a 'match_all'. This is not the desired
behavior in the MLT API if all the fields in a document are unsupported.
If that happens today we return all documents but the one MLT is
executed on.

Closes elastic#3453
@s1monw s1monw closed this as completed in 7f0115b Aug 7, 2013
s1monw added a commit that referenced this issue Aug 7, 2013
Today due the optimizations in the boolean query builder we adjust
a pure negative query with a 'match_all'. This is not the desired
behavior in the MLT API if all the fields in a document are unsupported.
If that happens today we return all documents but the one MLT is
executed on.

Closes #3453
mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015
Today due the optimizations in the boolean query builder we adjust
a pure negative query with a 'match_all'. This is not the desired
behavior in the MLT API if all the fields in a document are unsupported.
If that happens today we return all documents but the one MLT is
executed on.

Closes elastic#3453
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant