Skip to content

Remove LegacyCTRAL from MetadataIndexTemplateService#86459

Merged
DaveCTurner merged 1 commit intoelastic:masterfrom
DaveCTurner:2022-05-05-LegacyCTRAL-MetadataIndexTemplateService
May 5, 2022
Merged

Remove LegacyCTRAL from MetadataIndexTemplateService#86459
DaveCTurner merged 1 commit intoelastic:masterfrom
DaveCTurner:2022-05-05-LegacyCTRAL-MetadataIndexTemplateService

Conversation

@DaveCTurner
Copy link
Copy Markdown
Member

@DaveCTurner DaveCTurner commented May 5, 2022

Relates #83784, #86017

@DaveCTurner DaveCTurner added >non-issue :Data Management/Indices APIs DO NOT USE. Use ":Distributed/Indices APIs" or ":StorageEngine/Templates" instead. v8.3.0 labels May 5, 2022
@DaveCTurner DaveCTurner requested a review from dakrone May 5, 2022 10:39
@elasticmachine elasticmachine added the Team:Data Management (obsolete) DO NOT USE. This team no longer exists. label May 5, 2022
@elasticmachine
Copy link
Copy Markdown
Collaborator

elasticmachine commented May 5, 2022

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

@DaveCTurner
Copy link
Copy Markdown
Member Author

DaveCTurner commented May 5, 2022

Not quite as elegant as #86446 - these tasks involve various components of a Metadata and in several places we need to rebuild a Metadata for validation. It's probably possible to push this down to a lower level but it'd be a much bigger change so I'm just sticking with iterating on the ClusterState here.

@DaveCTurner DaveCTurner mentioned this pull request May 5, 2022
Closed
16 tasks
dakrone
dakrone approved these changes May 5, 2022
View reviewed changes
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

@DaveCTurner DaveCTurner merged commit 3ef46b0 into elastic:master May 5, 2022
@DaveCTurner DaveCTurner deleted the 2022-05-05-LegacyCTRAL-MetadataIndexTemplateService branch May 5, 2022 14:52
@DaveCTurner
Copy link
Copy Markdown
Member Author

DaveCTurner commented May 5, 2022

Thanks Lee

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

Reviewers

@dakrone dakrone dakrone approved these changes

Assignees

No one assigned

Labels

:Data Management/Indices APIs DO NOT USE. Use ":Distributed/Indices APIs" or ":StorageEngine/Templates" instead. >non-issue Team:Data Management (obsolete) DO NOT USE. This team no longer exists. v8.3.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@DaveCTurner @elasticmachine @dakrone