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: Rebase for #100782 #101248

Closed
wants to merge 2 commits into from
Closed

ESQL: Rebase for #100782 #101248

wants to merge 2 commits into from

Conversation

costin
Copy link
Member

@costin costin commented Oct 24, 2023

See #100782 - this PR just picked up the changes and rebased them on the latest main to remove the weird compilation errors.

astefan and others added 2 commits October 23, 2023 17:59
- properly accept a limit after mv_expand when there is also a second
limit before the mv_expand

Address feedback

Update docs/changelog/100782.yaml

Change the optimization rule
Add a default TopN for cases when there is only a sort at Lucene level
@costin costin added the :Analytics/ES|QL AKA ESQL label Oct 24, 2023
@costin costin requested a review from astefan October 24, 2023 03:29
@elasticsearchmachine elasticsearchmachine added v8.12.0 Team:QL (Deprecated) Meta label for query languages team labels Oct 24, 2023
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-ql (Team:QL)

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/elasticsearch-esql (:Query Languages/ES|QL)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/ES|QL AKA ESQL Team:QL (Deprecated) Meta label for query languages team v8.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants