Skip to content

Increase default write queue size#59464

Merged
Tim-Brooks merged 4 commits intoelastic:7.xfrom
Tim-Brooks:increase_write_7x
Jul 14, 2020
Merged

Increase default write queue size#59464
Tim-Brooks merged 4 commits intoelastic:7.xfrom
Tim-Brooks:increase_write_7x

Conversation

@Tim-Brooks
Copy link
Copy Markdown
Contributor

@Tim-Brooks Tim-Brooks commented Jul 13, 2020

This commit increases the default write queue size to 10000. This is to
allow a greater number of pending indexing requests. This work is safe
as we have added additional memory limits. Relates to #59263.

This commit increases the default write queue size to 4000. This is to
allow a greater number of pending indexing requests. This work is safe
as we have added additional memory limits.
@Tim-Brooks Tim-Brooks added >breaking :Distributed/CRUD A catch all label for issues around indexing, updating and getting a doc by id. Not search. v7.9.0 labels Jul 13, 2020
@Tim-Brooks Tim-Brooks requested a review from ywelsch July 13, 2020 20:32
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed (:Distributed/CRUD)

@elasticmachine elasticmachine added the Team:Distributed Meta label for distributed team. label Jul 13, 2020
@Tim-Brooks
Copy link
Copy Markdown
Contributor Author

After this is merged, it needs to be forward ported to 8.0 without the release notes. I have added a meta issue for that.

Copy link
Copy Markdown
Contributor

@ywelsch ywelsch left a comment

Choose a reason for hiding this comment

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

This is missing the actual change to the setting.

@Tim-Brooks Tim-Brooks requested a review from ywelsch July 14, 2020 15:18
@Tim-Brooks Tim-Brooks requested a review from ywelsch July 14, 2020 15:46
Copy link
Copy Markdown
Contributor

@ywelsch ywelsch left a comment

Choose a reason for hiding this comment

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

LGTM

@Tim-Brooks Tim-Brooks merged commit a46e5e0 into elastic:7.x Jul 14, 2020
@Tim-Brooks
Copy link
Copy Markdown
Contributor Author

This is currently labelled breaking as it changes a major default setting - the WRITE queue size from 200->10,000.

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

Labels

>breaking :Distributed/CRUD A catch all label for issues around indexing, updating and getting a doc by id. Not search. Team:Distributed Meta label for distributed team. v7.9.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants