Skip to content

[9.3] Fix StreamingXContentResponseIT#testAbort (#144473)#144560

Merged
elasticsearchmachine merged 3 commits intoelastic:9.3from
DaveCTurner:backport/9.3/pr-144473
Mar 23, 2026
Merged

[9.3] Fix StreamingXContentResponseIT#testAbort (#144473)#144560
elasticsearchmachine merged 3 commits intoelastic:9.3from
DaveCTurner:backport/9.3/pr-144473

Conversation

@DaveCTurner
Copy link
Copy Markdown
Member

Backports the following commits to 9.3:

Silly test mistake: the throttle was supposed to limit the number of
fragments in the queue, but was closing the throttling ref after
submission rather than transmission so that the queue could grow without
bound sometimes.

It's been like this since the test was introduced in elastic#111933, but it
only started failing recently. I suspect until the performance
improvement in elastic#139988 landed it couldn't actually submit enough tasks
at once to cause a failure.

Closes elastic#143644
@DaveCTurner DaveCTurner added :Distributed/Network Http and internode communication implementations >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:Distributed Meta label for distributed team. labels Mar 19, 2026
@elasticsearchmachine elasticsearchmachine merged commit ff4f7f0 into elastic:9.3 Mar 23, 2026
36 checks passed
@DaveCTurner DaveCTurner deleted the backport/9.3/pr-144473 branch March 23, 2026 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Distributed/Network Http and internode communication implementations Team:Distributed Meta label for distributed team. >test Issues or PRs that are addressing/adding tests v9.3.3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants