-
Notifications
You must be signed in to change notification settings - Fork 25k
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
NullPointerException with bool nested query #45933
Comments
Pinging @elastic/es-search |
Thanks @dvandersluis for this bug report. Would you be able to provide a full reproduction of the issue, including the mappings and documents that were defined? I tried to run the search request you provided on 7.3.1, but was not able to reproduce the |
Do you have a way I can share them with you? The index is 2.7GB so I'm not exactly sure how I would do that. I did notice that if I played around with the filters I could filter down the result set to a point where it does work so that does suggest to me that the NPE is dependent on what data is being searched. |
We were having the same issue with one of our indexes after upgrade to 7.3.1. So we had to downgrade to 7.1.1 version. |
I think I managed to reproduce the error locally by ingesting a larger number of documents. I have an idea of what the issue is and hope to put up a PR soon. @dvandersluis once the PR is up, it would be really helpful if you tried out the fix on your data to confirm it resolves the error you're seeing (assuming it's not too inconvenient). |
@jtibshirani thanks! I would love to help test it out, if you can email me to help me figure out how to get your patch up working with our data! |
FWIW the index in question has 32927960 docs. |
I merged a fix that will be available in the next 7.3 and 7.4 releases. If you would like to test out the fix now instead of waiting for a release, you can check out the Thanks again @dvandersluis for helping us to track down this bug. |
@jtibshirani thank you! We have our ES cluster managed so I'm not sure if I can build a version from source on it but I'll see what I can do. Any idea when the next 7.3 release will be? |
Sounds good, I will close issue for now and we can re-open if you notice more errors. We don't communicate release dates, but it should be available fairly soon! |
@jtibshirani we updated to 7.3.2 today and I'm monitoring but it appears to have fixed our issue! Thanks! |
Elasticsearch version (
bin/elasticsearch --version
):Plugins installed: None
JVM version (
java -version
):OS version (
uname -a
if on a Unix-like system):(Ubuntu 18.04.3 via Cloud66)
Description of the problem including expected versus actual behavior:
I'm experiencing NullPointerExceptions with this query (simplified but still causes the NPE):
I believe this may be related to #43497 although my query is not using
min_score
. The stacktrace looks similar to me.Stacktrace:
The text was updated successfully, but these errors were encountered: