Skip to content

Conversation

DaveCTurner
Copy link
Contributor

We have this somewhat-complex pattern in 3 places already, and #96279 will introduce a couple more, so this commit extracts it as a dedicated utility.

Relates #92987
Relates #93484

We have this somewhat-complex pattern in 3 places already, and elastic#96279
will introduce a couple more, so this commit extracts it as a dedicated
utility.

Relates elastic#92987
Relates elastic#93484
@DaveCTurner DaveCTurner added :Distributed Indexing/Distributed A catch all label for anything in the Distributed Indexing Area. Please avoid if you can. >refactoring v8.9.0 labels May 26, 2023
@elasticsearchmachine elasticsearchmachine added the Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. label May 26, 2023
@elasticsearchmachine
Copy link
Collaborator

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

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.

Thanks for all the comments, really helpful.

@DaveCTurner DaveCTurner merged commit 2b0d6e6 into elastic:main May 26, 2023
@DaveCTurner DaveCTurner deleted the 2023-05-26-CancellableFanOut branch May 26, 2023 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed Indexing/Distributed A catch all label for anything in the Distributed Indexing Area. Please avoid if you can. >refactoring Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. v8.9.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants