Skip to content

[WIP] Various Speedups to Cluster State Updates (ILM + Allocation) #79692

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

Conversation

original-brownbear
Copy link
Contributor

This contains a number of fixes/speedups to the allocation code as well as to ILM. This will be broken down into multiple pieces, some of which need quite a bit of polishing still.
That said, this branch speeds up certain cluster state updates massively. E.g. rolling over a data stream when there's already 30k indices present in a cluster goes from ~5s in our benchmark environment to ~1s.
The fixes to ILM in this PR effectively speed up every cluster state update, the larger the cluster the more the speedup and make ILM close to a noop as long as there's nothing for it to execute.
Memory consumption is also reduced to less than 1G of heap for 30k indices, from ~4G of heap in current 7.16 for the same cluster state.

I will start breaking things out of this PR one by one, opening PRs to master. This draft is mainly for keeping track of the work in in and some discussion.

@arteam arteam added v7.16.2 and removed v7.16.1 labels Dec 13, 2021
@mark-vieira mark-vieira changed the base branch from 7.16 to 7.17 December 16, 2021 14:40
@bpintea bpintea added v7.16.4 and removed v7.16.3 labels Jan 10, 2022
@arteam arteam added v7.17.3 and removed v7.17.2 labels Mar 28, 2022
@original-brownbear original-brownbear removed WIP :Distributed Coordination/Allocation All issues relating to the decision making around placing a shard (both master logic & on the nodes) :Data Management/ILM+SLM Index and Snapshot lifecycle management v7.16.4 v7.17.3 labels Jul 28, 2022
@original-brownbear
Copy link
Contributor Author

closing here, I merged what I wanted from this to mainline now

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

Successfully merging this pull request may close these issues.

8 participants