Skip to content

Test fix - release ByteBufs in Netty4HttpRequestBodyStreamTests#113161

Merged
mhl-b merged 1 commit intoelastic:mainfrom
mhl-b:test-fix-testEnqueueChunksBeforeRequest
Sep 19, 2024
Merged

Test fix - release ByteBufs in Netty4HttpRequestBodyStreamTests#113161
mhl-b merged 1 commit intoelastic:mainfrom
mhl-b:test-fix-testEnqueueChunksBeforeRequest

Conversation

@mhl-b
Copy link
Copy Markdown
Contributor

@mhl-b mhl-b commented Sep 19, 2024

Release ref counted ByteBufs in Netty4HttpRequestBodyStreamTests.

Closes #113159

@mhl-b mhl-b added >test Issues or PRs that are addressing/adding tests :Distributed/Network Http and internode communication implementations Team:Distributed Meta label for distributed team. v8.16.0 v9.0.0 labels Sep 19, 2024
@mhl-b mhl-b requested a review from Tim-Brooks September 19, 2024 05:57
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed (Team:Distributed)

@mhl-b mhl-b requested a review from ywangd September 19, 2024 06:16
Copy link
Copy Markdown
Member

@ywangd ywangd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mhl-b mhl-b merged commit 8e9e653 into elastic:main Sep 19, 2024
Tim-Brooks pushed a commit to Tim-Brooks/elasticsearch that referenced this pull request Sep 19, 2024
Tim-Brooks added a commit that referenced this pull request Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Network Http and internode communication implementations Team:Distributed Meta label for distributed team. >test Issues or PRs that are addressing/adding tests v8.16.0 v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] SimpleNetty4TransportTests testSendRandomRequests failing

4 participants