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.

Sign up for GitHub

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

Jump to bottom

Set queue sizes by default on bulk/index thread pools #3888

Closed
kimchy opened this issue Oct 11, 2013 · 0 comments
Closed

Set queue sizes by default on bulk/index thread pools #3888

kimchy opened this issue Oct 11, 2013 · 0 comments
Labels
>enhancement v0.90.6 v1.0.0.Beta1

Comments

@kimchy
Copy link
Member

kimchy commented Oct 11, 2013

Now that we properly fixed the ability to set the queue size on the index / bulk thread pool, we should actually set them to a somehow reasonable value to protect from users potentially overflowing our system.

I suggest defaults to be 50 for bulk, and 200 for indexing.

Also, set the thread pool for get, which we should set (in a similar value to a "read" queue size we have today).
@s1monw s1monw mentioned this issue Oct 11, 2013
Closed
@kimchy kimchy closed this as completed in 420b339 Oct 12, 2013
kimchy added a commit that referenced this issue Oct 12, 2013
@kimchy
Set queue sizes by default on bulk/index thread pools
a4bbf2e 
Now that we properly fixed the ability to set the queue size on the index / bulk thread pool, we should actually set them to a somehow reasonable value to protect from users potentially overflowing our system.

I suggest defaults to be 50 for bulk, and 200 for indexing.

Also, set the thread pool for get, which we should set (in a similar value to a "read" queue size we have today).
closes #3888
kimchy added a commit that referenced this issue Oct 12, 2013
@kimchy
Set queue sizes by default on bulk/index thread pools
81d5ecf 
Now that we properly fixed the ability to set the queue size on the index / bulk thread pool, we should actually set them to a somehow reasonable value to protect from users potentially overflowing our system.

I suggest defaults to be 50 for bulk, and 200 for indexing.

Also, set the thread pool for get, which we should set (in a similar value to a "read" queue size we have today).
closes #3888
mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015
@kimchy
Set queue sizes by default on bulk/index thread pools
584547b 
Now that we properly fixed the ability to set the queue size on the index / bulk thread pool, we should actually set them to a somehow reasonable value to protect from users potentially overflowing our system.

I suggest defaults to be 50 for bulk, and 200 for indexing.

Also, set the thread pool for get, which we should set (in a similar value to a "read" queue size we have today).
closes elastic#3888
mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015
@kimchy
Set queue sizes by default on bulk/index thread pools
12f2b7e 
Now that we properly fixed the ability to set the queue size on the index / bulk thread pool, we should actually set them to a somehow reasonable value to protect from users potentially overflowing our system.

I suggest defaults to be 50 for bulk, and 200 for indexing.

Also, set the thread pool for get, which we should set (in a similar value to a "read" queue size we have today).
closes elastic#3888
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
>enhancement v0.90.6 v1.0.0.Beta1
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@kimchy