Skip to content

Allocate searchable snapshots with the balancer#54889

Merged
DaveCTurner merged 2 commits intoelastic:masterfrom
DaveCTurner:2020-04-07-allocate-searchable-snapshots-with-balancer
Apr 8, 2020
Merged

Allocate searchable snapshots with the balancer#54889
DaveCTurner merged 2 commits intoelastic:masterfrom
DaveCTurner:2020-04-07-allocate-searchable-snapshots-with-balancer

Conversation

@DaveCTurner
Copy link
Copy Markdown
Member

Today the shards of searchable snapshots are allocated with a naive
ExistingShardsAllocator which selects the first valid node for each shard.
Thanks to #54729 we can now allow these shards to fall through to the balanced
shards allocator so that they are allocated in a more balanced fashion.

Relates #50999

Today the shards of searchable snapshots are allocated with a naive
`ExistingShardsAllocator` which selects the first valid node for each shard.
Thanks to elastic#54729 we can now allow these shards to fall through to the balanced
shards allocator so that they are allocated in a more balanced fashion.

Relates elastic#50999
@DaveCTurner DaveCTurner added >enhancement :Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs v8.0.0 v7.8.0 labels Apr 7, 2020
@DaveCTurner DaveCTurner requested a review from tlrx April 7, 2020 13:09
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed (:Distributed/Snapshot/Restore)

Copy link
Copy Markdown
Member

@tlrx tlrx left a comment

Choose a reason for hiding this comment

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

LGTM

@DaveCTurner
Copy link
Copy Markdown
Member Author

@elasticmachine update branch

@DaveCTurner DaveCTurner merged commit a5ec204 into elastic:master Apr 8, 2020
@DaveCTurner DaveCTurner deleted the 2020-04-07-allocate-searchable-snapshots-with-balancer branch April 8, 2020 09:02
DaveCTurner added a commit that referenced this pull request Apr 8, 2020
Today the shards of searchable snapshots are allocated with a naive
`ExistingShardsAllocator` which selects the first valid node for each shard.
Thanks to #54729 we can now allow these shards to fall through to the balanced
shards allocator so that they are allocated in a more balanced fashion.

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

Labels

:Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs >enhancement v7.8.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants