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

[8.11] ESQL: Add warning header when default LIMIT is applied (#100894) #101054

Merged

Conversation

luigidellaquila
Copy link
Contributor

Backports the following commits to 8.11:

Fixes elastic#100555

ESQL [by default](elastic#99816)
adds an implicit `LIMIT 500` to queries that do not define a limit.
Since this can be confusing for the end user, with this PR we also add a
warning to the response, making this default clear and explicit.
@luigidellaquila luigidellaquila added :Analytics/ES|QL AKA ESQL >non-issue auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:QL (Deprecated) Meta label for query languages team labels Oct 18, 2023
@github-actions
Copy link
Contributor

Documentation preview:

@elasticsearchmachine elasticsearchmachine merged commit c23a46b into elastic:8.11 Oct 18, 2023
@luigidellaquila luigidellaquila deleted the backport/8.11/pr-100894 branch October 18, 2023 12:28
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 >non-issue Team:QL (Deprecated) Meta label for query languages team v8.11.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants