Skip to content

Add downsampling_method to the data stream lifecycle#5594

Merged
pquentin merged 5 commits intomainfrom
downsampling/add-sampling-method-in-dlm
Dec 5, 2025
Merged

Add downsampling_method to the data stream lifecycle#5594
pquentin merged 5 commits intomainfrom
downsampling/add-sampling-method-in-dlm

Conversation

@gmarouli
Copy link
Contributor

In elastic/elasticsearch#137023 we add the sampling method also in the data stream lifecycle.

This first two commits of this PR do the following:

This PR is work in progress until #5593 is validated an merged.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 31, 2025

Following you can find the validation changes against the target branch for the APIs.

API Status Request Response
cluster.get_component_template 🔴 → 🟢 15/15 14/15 → 15/15
cluster.put_component_template 🔴 33/38 → 34/38 38/38
indices.get_index_template 🔴 → 🟢 24/24 23/24 → 24/24
indices.put_data_lifecycle 🔴 → 🟢 6/7 → 7/7 7/7
indices.put_index_template 🔴 146/171 → 147/171 171/171

You can validate these APIs yourself by using the make validate target.

Copy link
Contributor

@nielsbauman nielsbauman left a comment

Choose a reason for hiding this comment

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

These changes also LGTM :)

@pquentin pquentin marked this pull request as ready for review December 5, 2025 11:20
@pquentin pquentin changed the title [WIP] Add downsampling method to the data stream lifecycle Add downsampling_method to the data stream lifecycle Dec 5, 2025
Copy link
Member

@pquentin pquentin left a comment

Choose a reason for hiding this comment

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

Thanks! LGTM.

@pquentin pquentin merged commit 12d3761 into main Dec 5, 2025
10 checks passed
@pquentin pquentin deleted the downsampling/add-sampling-method-in-dlm branch December 5, 2025 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-backport This pull request should not be backported specification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants