Skip to content
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

Rename INDEX_ROUTING_PREFER to TIER_PREFERENCE #78411

Merged
merged 8 commits into from
Sep 29, 2021

Conversation

joegallo
Copy link
Contributor

Related to #76147, this is a cleanup and refactoring PR that just gets some groundwork out of the way -- here as a separate PR to keep overall noise down.

Reviewing commit by commit is probably easiest.

@joegallo joegallo added >non-issue :Data Management/ILM+SLM Index and Snapshot lifecycle management v8.0.0 v7.16.0 labels Sep 28, 2021
@elasticmachine elasticmachine added the Team:Data Management Meta label for data/management team label Sep 28, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

@joegallo joegallo force-pushed the data-tier-preference-rename branch from 25dd5e6 to f740e28 Compare September 28, 2021 19:42
No need to poke into the implementation and re-implement this locally
in terms of the set itself
In the end the version from the request or templates will be used
anyway (instead of whatever we say here), so it seems more intuitive
to return null as "don't care" rather than "hot (but I know you'll
ignore this)".
@joegallo joegallo force-pushed the data-tier-preference-rename branch from f740e28 to 5d4a0bf Compare September 28, 2021 20:37
Copy link
Member

@dakrone dakrone left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/ILM+SLM Index and Snapshot lifecycle management >non-issue Team:Data Management Meta label for data/management team v7.16.0 v8.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants