Skip to content

Conversation

@DaveCTurner
Copy link
Contributor

Today we do not have a test that verifies the Netty HTTP pipeline
interacts properly with the incremental bulk handling service and splits
requests when the watermark is hit. This commit adds such a test.

Today we do not have a test that verifies the Netty HTTP pipeline
interacts properly with the incremental bulk handling service and splits
requests when the watermark is hit. This commit adds such a test.
@DaveCTurner DaveCTurner added >test Issues or PRs that are addressing/adding tests :Distributed Indexing/CRUD A catch all label for issues around indexing, updating and getting a doc by id. Not search. :Distributed Coordination/Network Http and internode communication implementations v9.1.0 labels Apr 23, 2025
@elasticsearchmachine elasticsearchmachine added Team:Distributed Coordination Meta label for Distributed Coordination team Team:Distributed Indexing Meta label for Distributed Indexing team labels Apr 23, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Contributor

@henningandersen henningandersen left a comment

Choose a reason for hiding this comment

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

LGTM.

@DaveCTurner DaveCTurner added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Apr 23, 2025
@elasticsearchmachine elasticsearchmachine merged commit 85a87e7 into elastic:main Apr 23, 2025
17 checks passed
@DaveCTurner DaveCTurner deleted the 2025/04/23/netty4-incremental-bulk-test branch April 23, 2025 15:43
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!) :Distributed Coordination/Network Http and internode communication implementations :Distributed Indexing/CRUD A catch all label for issues around indexing, updating and getting a doc by id. Not search. Team:Distributed Coordination Meta label for Distributed Coordination team Team:Distributed Indexing Meta label for Distributed Indexing team >test Issues or PRs that are addressing/adding tests v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants