Skip to content

Using DataStream::getEffectiveSettings #127282

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

Merged

Conversation

masseyke
Copy link
Member

This builds on #126947, by using the DataStream::getEffectiveSettings and DataStream::getEffectiveIndexTemplate methods introduced in that PR.
As of this PR, no externally-visible behavior is different. Rollover will now use the effective settings from the data stream, but since the settings actually stored on the data stream area always empty, the result is that the template settings continue to be used. Similarly, the settings used in the GetDataStreamAction will be the effective settings but will appear unchanged since the data stream settings are always empty.
A follow-up PR will introduce a transport action to set the data stream's settings.

@masseyke masseyke marked this pull request as ready for review April 23, 2025 19:25
@elasticsearchmachine elasticsearchmachine added the Team:Data Management Meta label for data/management team label Apr 23, 2025
@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Contributor

@lukewhiting lukewhiting left a comment

Choose a reason for hiding this comment

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

Change looks good to me 👍🏻

Copy link
Member

@jbaiera jbaiera left a comment

Choose a reason for hiding this comment

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

LGTM!

@masseyke masseyke merged commit 7ddc8d9 into elastic:main Apr 25, 2025
17 checks passed
@masseyke masseyke deleted the using-data-stream-get-effective-settings branch April 25, 2025 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/Data streams Data streams and their lifecycles >non-issue Team:Data Management Meta label for data/management team v9.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants