Skip to content

WaitGroup: spawn into an explicit Fiber::ExecutionContext #15643

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ysbaddaden
Copy link
Contributor

closes #15642

@yxhuvud
Copy link
Contributor

yxhuvud commented Apr 8, 2025

One could perhaps take a step back and consider if the marriage between wait groups and contexts should live in either of these or if it should be an entity of its own. As a concept it is getting pretty close to the concept of nurseries in the structured concurrency way of thinking.

Where are we going, in the greater path toward more sophisticated ways of setting up execution?

@ysbaddaden
Copy link
Contributor Author

IMO the nursery-like methods (.wait(&) and .spawn(&)) are the issue. Not the ability to specify a spawn context.

@ysbaddaden ysbaddaden added the pr:on-hold-do-not-merge This PR is on hold. Do not merge. label Apr 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Review
Development

Successfully merging this pull request may close these issues.

Add ExecutionContext support to WaitGroup
4 participants