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

Memory pre-allocation with size attribute #6463

Closed
marcelopaesrech opened this issue Jun 11, 2014 · 4 comments
Closed

Memory pre-allocation with size attribute #6463

marcelopaesrech opened this issue Jun 11, 2014 · 4 comments
Assignees

Comments

@marcelopaesrech
Copy link

When I do a filter or query, and set the size valeu to a huge value (like 10000000), the memory consuption is increased a lot even though the final result be 10 documents.

If I do the same thing with a huge amount of users the memory is increased so fast that the nodes would become bottlenecks (sometimes a node shutdowns itself).

I solved that changing the size attribute to a low value, but I think it would be better if the ES deal with that in a better way, avoiding bad programmers to shutdown the server.

Regards.

@marcelopaesrech
Copy link
Author

@jpountz jpountz self-assigned this Jun 13, 2014
@jpountz
Copy link
Contributor

jpountz commented Jun 13, 2014

Would it work for you if we allowed to configure a maximum size?

@marcelopaesrech
Copy link
Author

Hi @jpountz,

Thanks for your reply.

I don't know if that would be the best solution. There is already a Size parameter for query/filter.

The problem is the programmer is responsible to do that, and some times he will not understand what big size parameter in query/filter would cause in production environment.

A maximum size in the entire cluster or index would help avoiding cluster shutdown. But I think it would be better count how many documents are available and then cut off pre-allocated memory that would not be used.

Regards.

@clintongormley
Copy link
Contributor

Closing in favour of #8080

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

No branches or pull requests

3 participants