-
Notifications
You must be signed in to change notification settings - Fork 24.6k
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
Reduce memory usage in top children query #8165
Conversation
@kire321 Thanks for addressing this unnecessary usage of memory! The change looks good. Can you sign the CLA: http://www.elasticsearch.org/contributor-agreement/ ? Then this change can get merged in. |
I did sign the CLA... Do I need to associate the email I used to sign it with my github account? |
@kire321 As far as I know there is no need to for the email to be in sync with your GH account. |
@martijnvg Is there anything I need to do to help the CLA get through? |
@kire321 The CLA is fine, I'll pull your change in. |
@kire321 I pushed your change and thanks for brining this up. |
Closes #8160