Skip to content

Add set_priority action to ILM#37397

Merged
jakelandis merged 9 commits intoelastic:masterfrom
jakelandis:ilm_index_priority
Jan 17, 2019
Merged

Add set_priority action to ILM#37397
jakelandis merged 9 commits intoelastic:masterfrom
jakelandis:ilm_index_priority

Conversation

@jakelandis
Copy link
Copy Markdown
Contributor

This commit adds a set_priority action to the hot, warm, and cold
phases for an ILM policy. This action sets the index.priority
on the managed index to allow different priorities between the
hot, warm, and cold recoveries.

This commit also includes the HLRC and documentation changes.

closes #36905

This commit adds a set_priority action to the hot, warm, and cold
phases for an ILM policy. This action sets the `index.priority`
on the managed index to allow different priorities between the
hot, warm, and cold recoveries.

This commit also includes the HLRC and documentation changes.

closes elastic#36905
@jakelandis jakelandis added >enhancement v7.0.0 :Data Management/ILM+SLM DO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead. v6.7.0 labels Jan 13, 2019
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-features

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

Thanks for opening this @jakelandis, I left some comments

Copy link
Copy Markdown
Contributor

@AthenaEryma AthenaEryma left a comment

Choose a reason for hiding this comment

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

Overall looks good aside from the stuff Lee brought up, these are mostly nitpicks.

@jakelandis
Copy link
Copy Markdown
Contributor Author

@dakrone @gwbrown Thanks for the reviews! changes implemented and ready for re-review. The only substantial changes are on commit 478891f (support null to unset priority) , the rest of the changes are minor/nits.

Copy link
Copy Markdown
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, I left 3 minor comments

Copy link
Copy Markdown
Contributor

@AthenaEryma AthenaEryma left a comment

Choose a reason for hiding this comment

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

LGTM to me as well, once Lee's remaining comments are addressed and CI is happy.

@jakelandis
Copy link
Copy Markdown
Contributor Author

@elasticmachine test this please

@jakelandis
Copy link
Copy Markdown
Contributor Author

@elasticmachine run gradle build tests 1
@elasticmachine run gradle build tests 2

@jakelandis
Copy link
Copy Markdown
Contributor Author

@elasticmachine run gradle build tests 1

@jakelandis jakelandis merged commit 587034d into elastic:master Jan 17, 2019
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Jan 17, 2019
…-response-header-performance

* elastic/master:
  Remove Redundant RestoreRequest Class (elastic#37535)
  Create specific exception for when snapshots are in progress (elastic#37550)
  Mute UnicastZenPingTests#testSimplePings
  [DOCS] Adds size limitation to the get datafeeds APIs (elastic#37578)
  Fix assertion at end of forceRefreshes (elastic#37559)
  Propagate Errors in executors to uncaught exception handler (elastic#36137)
  [DOCS] Adds limitation to the get jobs API (elastic#37549)
  Add set_priority action to ILM (elastic#37397)
  Make recovery source send operations non-blocking (elastic#37503)
  Allow field types to optimize phrase prefix queries (elastic#37436)
  Added fatal_exception field for ccr stats in monitoring mapping. (elastic#37563)
  Fix testRelocateWhileContinuouslyIndexingAndWaitingForRefresh (elastic#37560)
  Moved ccr integration to the package with other ccr integration tests.
  Mute TransportClientNodesServiceTests#testListenerFailures
  Decreased time out in test
  Fix erroneous docstrings for abstract bulk by scroll request (elastic#37517)
  SQL: Rename SQL type DATE to DATETIME (elastic#37395)
  Remove the AbstracLifecycleComponent constructor with Settings (elastic#37523)
jakelandis added a commit that referenced this pull request Jan 17, 2019
This commit adds a set_priority action to the hot, warm, and cold
phases for an ILM policy. This action sets the `index.priority`
on the managed index to allow different priorities between the
hot, warm, and cold recoveries.

This commit also includes the HLRC and documentation changes.

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

Labels

:Data Management/ILM+SLM DO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead. >enhancement v6.7.0 v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ILM] rollover + settings.index.priority

5 participants