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
Fix percent_terms_to_match
#7754
Fix percent_terms_to_match
#7754
Conversation
The parameter `percent_terms_to_match` (number of terms that must match in the generated query) was wrongly set to the top level boolean query. This would lead to zero or all results type of situations. This commit ensures that the parameter is indeed applied to the query of generated terms.
LGTM Yet - I think we should really support the common syntax for minimum should match and use |
That would definitely be nice. However, I'd prefer to do it in another commit. This commit fixes a bug introduced when switching to using the TVs API (https://github.com/elasticsearch/elasticsearch/pull/7014/files#diff-8790389477fb6c598af492a8e673f858R153), which is a 1.4beta release. |
yeah lets open a different issue. so LGTM :) |
The parameter `percent_terms_to_match` (percentage of terms that must match in the generated query) was wrongly set to the top level boolean query. This would lead to zero or all results type of situations. This commit ensures that the parameter is indeed applied to the query of generated terms. Closes #7754
The parameter `percent_terms_to_match` (percentage of terms that must match in the generated query) was wrongly set to the top level boolean query. This would lead to zero or all results type of situations. This commit ensures that the parameter is indeed applied to the query of generated terms. Closes #7754
percent_terms_to_match
The parameter `percent_terms_to_match` (percentage of terms that must match in the generated query) was wrongly set to the top level boolean query. This would lead to zero or all results type of situations. This commit ensures that the parameter is indeed applied to the query of generated terms. Closes elastic#7754
The parameter
percent_terms_to_match
(number of terms that must match in thegenerated query) was wrongly set to the top level boolean query. This would
lead to zero or all results type of situations. This commit ensures that the
parameter is indeed applied to the query of generated terms.