Skip to content
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

ESQL: mv_expand pushes down a limit copy and keeps the limit after it untouched (#100782) #101268

Merged
merged 1 commit into from
Oct 24, 2023

Conversation

astefan
Copy link
Contributor

@astefan astefan commented Oct 24, 2023

ESQL: mv_expand pushes down a limit copy and keeps the limit after it untouched (#100782)

  • allow mv_expand to push down limit and project past it
  • accept a limit after mv_expand when there is also a second limit before the mv_expand
  • adds a default TopN for cases when there is only a sort at Lucene level
  • adds OrderBy node type to the exceptions for duplicating the limit after mv_expand

(cherry picked from commit 4679b09)

… untouched (elastic#100782)

- allow mv_expand to push down limit and project past it
- accept a limit after mv_expand when there is also a second limit before the mv_expand
- adds a default TopN for cases when there is only a sort at Lucene level
- adds OrderBy node type to the exceptions for duplicating the limit after mv_expand

(cherry picked from commit 4679b09)
@astefan astefan added >bug backport auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Analytics/ES|QL AKA ESQL v8.11.0 v8.12.0 labels Oct 24, 2023
@elasticsearchmachine elasticsearchmachine merged commit 8f8ce7e into elastic:8.11 Oct 24, 2023
@astefan astefan deleted the 100782_8.11_backport branch October 24, 2023 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/ES|QL AKA ESQL auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >bug v8.11.0 v8.11.1 v8.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants