Skip to content

More accurate error message for datastream and alias#79027

Merged
ywangd merged 1 commit into
elastic:masterfrom
ywangd:datastream-alias-error-message
Oct 13, 2021
Merged

More accurate error message for datastream and alias#79027
ywangd merged 1 commit into
elastic:masterfrom
ywangd:datastream-alias-error-message

Conversation

@ywangd

@ywangd ywangd commented Oct 13, 2021

Copy link
Copy Markdown
Member

Alias for datastream is supported since #66163. Currently alias for the
backing indices are still not allowed. As a result, the error messages
are updated to reflect the latest status.

Relates: #58327

Alias for datastream is supported since elastic#66163. Currently alias for the
backing indices are still not allowed. As a result, the error messages
are updated to reflect the latest status.
@elasticmachine elasticmachine added the Team:Data Management (obsolete) DO NOT USE. This team no longer exists. label Oct 13, 2021
@elasticmachine

Copy link
Copy Markdown
Collaborator

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

@danhermann danhermann left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM pending successful CI. Thanks for fixing this, @ywangd!

@ywangd ywangd merged commit fff2cb1 into elastic:master Oct 13, 2021
ywangd added a commit to ywangd/elasticsearch that referenced this pull request Oct 13, 2021
Alias for datastream is supported since elastic#66163. Currently alias for the
backing indices are still not allowed. As a result, the error messages
are updated to reflect the latest status.
@elasticsearchmachine

Copy link
Copy Markdown
Collaborator

💚 Backport successful

Status Branch Result
7.x

elasticsearchmachine pushed a commit that referenced this pull request Oct 13, 2021
Alias for datastream is supported since #66163. Currently alias for the
backing indices are still not allowed. As a result, the error messages
are updated to reflect the latest status.

@martijnvg martijnvg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM2! Thanks for fixing.

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

Labels

>enhancement :StorageEngine/Data streams Data streams and their lifecycles Team:Data Management (obsolete) DO NOT USE. This team no longer exists. v7.16.0 v8.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants