Skip to content

Conversation

@dakrone
Copy link
Member

@dakrone dakrone commented Mar 31, 2021

This commits makes the shared_cache searchable snapshot cache size setting resolve to 0 for nodes
that to do have the data_frozen node role.

Tangentially related to #70341
Supercedes #70846

This only targets 7.x, because master will be handled by #71013

This commits makes the shared_cache searchable snapshot cache size setting resolve to 0 for nodes
that to do have the data_frozen node role.

Tangentially related to elastic#70341
Supercedes elastic#70846
@dakrone dakrone added :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs v7.13.0 labels Mar 31, 2021
@elasticmachine elasticmachine added the Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. label Mar 31, 2021
@elasticmachine
Copy link
Collaborator

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

Copy link
Member

@jasontedor jasontedor left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
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.

The message in HasFrozenCacheAllocationDecider.NO_FROZEN_CACHE should also be adapted as it is highly confusing now.

@dakrone
Copy link
Member Author

dakrone commented Apr 1, 2021

Thanks for bringing this up @ywelsch, I added a commit to change the message to clarify

@dakrone dakrone merged commit 7962e9a into elastic:7.x Apr 1, 2021
@dakrone dakrone deleted the make-cache-size-zero-7x branch April 1, 2021 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs >enhancement Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. v7.13.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants