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

Add Force Merge to hot lifecycle phase #43165

Closed
mud5150 opened this issue Jun 12, 2019 · 4 comments · Fixed by #52073
Closed

Add Force Merge to hot lifecycle phase #43165

mud5150 opened this issue Jun 12, 2019 · 4 comments · Fixed by #52073
Assignees
Labels
:Data Management/ILM+SLM Index and Snapshot lifecycle management >enhancement

Comments

@mud5150
Copy link

mud5150 commented Jun 12, 2019

Add Force Merge to hot lifecycle phase:
For time series data it makes sense that you may want to do a force merge on an index in the hot phase after rollover occurs. Seems pretty straight forward but if you want more information LMK.

@polyfractal polyfractal added the :Data Management/ILM+SLM Index and Snapshot lifecycle management label Jun 12, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features

@dakrone
Copy link
Member

dakrone commented Jun 12, 2019

For time series data it makes sense that you may want to do a force merge on an index in the hot phase after rollover occurs.

@mud5150 will doing a hot phase with a rollover action and then having a "warm" phase with no min_age set so that it performs the force merge immediately after the rollover happens work? You can do that currently. Is there a reason the force merge would need to happen in the hot phase?

@mud5150
Copy link
Author

mud5150 commented Jun 14, 2019

It could work, however in my particular instance all phases are in use. In my instance "hot" servers hold 28 days of data. Warm servers (slower disks) hold the rest of the year and still get a good amount of user queries. Cold nodes have slow disks and high ratio of storage to memory, use frozen indices, etc. It's not a problem for me to have curator force merge the index after the rollover happens, I just thought the above seems like a reasonable use case.

@dakrone
Copy link
Member

dakrone commented Jan 21, 2020

We discussed this and decided to allow the force merge action in the hot phase, with a small caveat. We will require that the rollover action also be present (since rollover will occur prior to the force merge).

@dakrone dakrone self-assigned this Feb 4, 2020
dakrone added a commit to dakrone/elasticsearch that referenced this issue Feb 7, 2020
This commit changes the `forcemerge` action to also be allowed in the `hot` phase for policies. The
forcemerge will occur after a rollover, and allows users to take advantage of higher disk speeds for
performing the force merge (on a separate node type, for example).

On caveat with this is that a `forcemerge` in the `hot` phase *MUST* be accompanied by a `rollover`
action. ILM validates policies to ensure this is the case.

Resolves elastic#43165
dakrone added a commit that referenced this issue Feb 7, 2020
* Allow forcemerge in the hot phase for ILM policies

This commit changes the `forcemerge` action to also be allowed in the `hot` phase for policies. The
forcemerge will occur after a rollover, and allows users to take advantage of higher disk speeds for
performing the force merge (on a separate node type, for example).

On caveat with this is that a `forcemerge` in the `hot` phase *MUST* be accompanied by a `rollover`
action. ILM validates policies to ensure this is the case.

Resolves #43165

* Use anyMatch instead of findAny in validation

* Make randomTimeseriesLifecyclePolicy single-pass
dakrone added a commit to dakrone/elasticsearch that referenced this issue Feb 7, 2020
* Allow forcemerge in the hot phase for ILM policies

This commit changes the `forcemerge` action to also be allowed in the `hot` phase for policies. The
forcemerge will occur after a rollover, and allows users to take advantage of higher disk speeds for
performing the force merge (on a separate node type, for example).

On caveat with this is that a `forcemerge` in the `hot` phase *MUST* be accompanied by a `rollover`
action. ILM validates policies to ensure this is the case.

Resolves elastic#43165

* Use anyMatch instead of findAny in validation

* Make randomTimeseriesLifecyclePolicy single-pass
dakrone added a commit that referenced this issue Feb 10, 2020
* Allow forcemerge in the hot phase for ILM policies

This commit changes the `forcemerge` action to also be allowed in the `hot` phase for policies. The
forcemerge will occur after a rollover, and allows users to take advantage of higher disk speeds for
performing the force merge (on a separate node type, for example).

On caveat with this is that a `forcemerge` in the `hot` phase *MUST* be accompanied by a `rollover`
action. ILM validates policies to ensure this is the case.

Resolves #43165

* Use anyMatch instead of findAny in validation

* Make randomTimeseriesLifecyclePolicy single-pass
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 >enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants