[Concurrency] Parent task cancellation must cancel task group itself. #80794
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Seems that during refactorings of child cancellations we somehow missed also cancelling the group itself. It seems we did not have good test coverage of the addTaskUnlessCancelled somehow and thus this slipped through. This adds a regression test for addTaskUnlessCancelled and fixes how we handle the cancellation effect in TaskStatus. resolves swiftlang#80789 resolves rdar://149177600
|
Refactored the |
This way we do the right thing always when cancelling the group; and we MAY visit the child tasks if we have to.
ktoso
force-pushed
the
wip-parent-cancel-must-cancel-group
branch
from
b3c3a2c to
94423da
Compare
|
@swift-ci please smoke test |
This was referenced Apr 14, 2025
al45tair
reviewed
Apr 14, 2025
test/Concurrency/Runtime/async_taskgroup_addUnlessCancelled.swift
Outdated
Show resolved
Hide resolved
al45tair
approved these changes
Apr 14, 2025
|
@swift-ci please smoke test |
ktoso
force-pushed
the
wip-parent-cancel-must-cancel-group
branch
from
7d662fa to
a9bcf39
Compare
|
@swift-ci please smoke test |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Seems that during refactorings of child cancellations we somehow missed
also cancelling the group itself. It seems we did not have good test
coverage of the addTaskUnlessCancelled somehow and thus this slipped
through.
This adds a regression test for addTaskUnlessCancelled and fixes how we
handle the cancellation effect in TaskStatus.
resolves #80789
resolves rdar://149177600