Skip to content

Conversation

@jasontedor
Copy link
Member

The index thread pool is no longer needed as its primary use-case for single-document indexing requests has been relieved now that single-document indexing requests are converted to bulk indexing requests (with a single document payload).

The index thread pool is no longer needed as its primary use-case for
single-document indexing requests has been relieved now that
single-document indexing requests are converted to bulk indexing
requests (with a single document payload).
@jasontedor jasontedor added review :Core/Infra/Core Core issues without another label v7.0.0 v6.3.0 labels Apr 16, 2018
@jasontedor jasontedor requested a review from bleskes April 16, 2018 18:19
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@jasontedor
Copy link
Member Author

A follow-up will remove this thread pool from master for 7.0.0 only.

Copy link
Contributor

@bleskes bleskes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thx.

@jasontedor jasontedor merged commit 1dd0fd4 into elastic:master Apr 17, 2018
jasontedor added a commit that referenced this pull request Apr 17, 2018
The index thread pool is no longer needed as its primary use-case for
single-document indexing requests has been relieved now that
single-document indexing requests are converted to bulk indexing
requests (with a single document payload).
@jasontedor jasontedor deleted the deprecate-index-thread-pool branch April 17, 2018 10:47
@jasontedor
Copy link
Member Author

Thanks @bleskes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

5 participants