Skip to content

[8.18] Aggs: Fix Filters agg cancellation flaky test (#130810)#130902

Merged
elasticsearchmachine merged 1 commit intoelastic:8.18from
ivancea:backport/8.18/pr-130810
Jul 9, 2025
Merged

[8.18] Aggs: Fix Filters agg cancellation flaky test (#130810)#130902
elasticsearchmachine merged 1 commit intoelastic:8.18from
ivancea:backport/8.18/pr-130810

Conversation

@ivancea
Copy link
Contributor

@ivancea ivancea commented Jul 9, 2025

Backports the following commits to 8.18:

Fixes elastic#130770

Both a bigger thread pool and not draining the semaphore permits were leading to failing tests sometimes because of blocked threads (Too many threads searching would end up draining all permits in parallel, and getting stuck).

The test was added in elastic#130452
@ivancea ivancea added :Analytics/Aggregations Aggregations >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 Jul 9, 2025
@elasticsearchmachine elasticsearchmachine merged commit d63d29c into elastic:8.18 Jul 9, 2025
16 checks passed
@ivancea ivancea deleted the backport/8.18/pr-130810 branch July 9, 2025 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/Aggregations Aggregations 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.5

2 participants