Skip to content

[8.x] ESQL: Fix limit task test (#117270)#117378

Merged
nik9000 merged 2 commits intoelastic:8.xfrom
nik9000:backport/8.x/pr-117270
Dec 2, 2024
Merged

[8.x] ESQL: Fix limit task test (#117270)#117378
nik9000 merged 2 commits intoelastic:8.xfrom
nik9000:backport/8.x/pr-117270

Conversation

@nik9000
Copy link
Copy Markdown
Member

@nik9000 nik9000 commented Nov 22, 2024

Backports the following commits to 8.x:

Fix a test for the task results when running the `LIMIT` operation. We
were releasing a few permits to get the query started. And when you
combine that with the page worth of permits that the test was releasing
we'd sometimes finish the entire limited query, stopping the task too
early to find a running task.

Closes elastic#107293
@nik9000 nik9000 added :Analytics/ES|QL AKA ESQL >test Issues or PRs that are addressing/adding tests auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) labels Nov 22, 2024
@nik9000 nik9000 merged commit f3a58b6 into elastic:8.x Dec 2, 2024
@nik9000 nik9000 deleted the backport/8.x/pr-117270 branch December 2, 2024 17:27
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 Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v8.18.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants