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
Query String caching could cause matched_filters not working #4371
Conversation
Looks good, the bug is indeed related to caching, so it would be great if the pull request title and comment would reflect that. Also, because its relating to caching, we can simply test for it. For example, index the docs, and execute the search requests with preference set to _primary, and execute it a couple of times, the second one should kick in the caching, and the bug. |
When searching with a query containing query_strings inside a bool query, the specified _name is randomly missing from the results due to caching. Closes elastic#4361. Closes elastic#4371.
@kimchy PR updated. You're right. Using _primary cause it failing at every run before the patch. |
look good, I would imply instead of duplicating the code, run it in a for loop and do it a couple of times? |
@kimchy done and pushed. Thanks for the review. |
When searching with a query containing query_strings inside a bool query, the specified _name is randomly missing from the results due to caching. Closes elastic#4361. Closes elastic#4371.
Thanks @simonw for the review. Related to elastic#4361 and elastic#4371.
Thanks @simonw for the review. Related to elastic#4361 and elastic#4371.
PR for #4361.
When searching with a query containing query_strings inside a bool query, the specified _name is randomly missing from the results due to caching.