-
Notifications
You must be signed in to change notification settings - Fork 25.6k
Add release note for #135051 #135475
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
Add release note for #135051 #135475
Conversation
Hi @smalyshev, I've created a changelog YAML for you. Note that since this PR is labelled |
Pinging @elastic/es-storage-engine (Team:StorageEngine) |
Pinging @elastic/core-docs (Team:Docs) |
docs/changelog/135051.yaml
Outdated
highlight: | ||
title: Mark Limit + MvExpand before remote Enrich as unsupported | ||
body: |- | ||
Due to the distribured nature of remote `ENRICH`, using a `LIMIT` command followed by a `MV_EXPAND` command before a remote `ENRICH` command |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typodistribured
Looks good to me. Cc @leemthompo, as this will affect the docs. We should consider adding a known issues for pre-9.1/8.19 versions, which will still return wrong results (even though CCS in ESQL was only in tech preview then and remote enrich was technically only useful for CCS queries) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Saw opportunity to clarify wording here :)
@alex-spies could you please create an issue for adding known issues for pre-9.1/8.19 versions? I'll get that assigned in a follow up |
Co-authored-by: Liam Thompson <[email protected]>
Co-authored-by: Liam Thompson <[email protected]>
* Add release note for elastic#135051
* Add release note for elastic#135051
sorry ignore, the PR didn't have the labels, but you backported manually :) |
Yeah I didn't think about backporting until later so the original one didn't have the labels. |
See also: #135051