Skip to content

Remove DlmAction and DlmStep framework#144414

Merged
dakrone merged 2 commits intoelastic:mainfrom
dakrone:dlm-remove-action-framework
Mar 17, 2026
Merged

Remove DlmAction and DlmStep framework#144414
dakrone merged 2 commits intoelastic:mainfrom
dakrone:dlm-remove-action-framework

Conversation

@dakrone
Copy link
Copy Markdown
Member

@dakrone dakrone commented Mar 17, 2026

This framework was behind a feature flag (and thus not used in production). We are moving to a different execution model for the frozen conversion.

Relates to #144248 (comment)

This framework was behind a feature flag (and thus not used in production). We are moving to a
different execution model for the frozen conversion.

Relates to elastic#144248 (comment)
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-storage-engine (Team:StorageEngine)

@dakrone dakrone enabled auto-merge (squash) March 17, 2026 16:21
@seanzatzdev
Copy link
Copy Markdown
Contributor

Any harm in leaving this around a little longer? Would make it a little easier to move the existing code around, though I guess it's not too big of a hassle

@dakrone
Copy link
Copy Markdown
Member Author

dakrone commented Mar 17, 2026

Any harm in leaving this around a little longer? Would make it a little easier to move the existing code around, though I guess it's not too big of a hassle

You should be able to look at a commit from ES prior to this one being merged to see the existing code, if that works for you?

Copy link
Copy Markdown
Contributor

@seanzatzdev seanzatzdev left a comment

Choose a reason for hiding this comment

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

lgtm

@dakrone dakrone merged commit cdaa5c7 into elastic:main Mar 17, 2026
37 checks passed
michalborek pushed a commit to michalborek/elasticsearch that referenced this pull request Mar 23, 2026
This framework was behind a feature flag (and thus not used in production). We are moving to a
different execution model for the frozen conversion.

Relates to elastic#144248 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants