Skip to content

cancelled/resumed->rescheduled transition#6913

Merged
crusaderky merged 4 commits intodask:mainfrom
crusaderky:cancelled_rescheduled
Aug 19, 2022
Merged

cancelled/resumed->rescheduled transition#6913
crusaderky merged 4 commits intodask:mainfrom
crusaderky:cancelled_rescheduled

Conversation

@crusaderky
Copy link
Copy Markdown
Collaborator

@crusaderky crusaderky commented Aug 18, 2022

Closes #6685

@crusaderky crusaderky force-pushed the cancelled_rescheduled branch from 62b7f11 to 134394d Compare August 18, 2022 19:02
@crusaderky crusaderky force-pushed the cancelled_rescheduled branch from 134394d to dd07146 Compare August 18, 2022 19:03
@crusaderky crusaderky requested a review from fjetter August 18, 2022 19:20
@crusaderky crusaderky self-assigned this Aug 18, 2022
@crusaderky crusaderky marked this pull request as ready for review August 18, 2022 19:30
Copy link
Copy Markdown
Member

@hendrikmakait hendrikmakait left a comment

Choose a reason for hiding this comment

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

LGTM, one suggestion for a docstring. Thanks @crusaderky!

@crusaderky crusaderky merged commit 3369af0 into dask:main Aug 19, 2022
@crusaderky crusaderky deleted the cancelled_rescheduled branch August 19, 2022 14:43
gjoseph92 pushed a commit to gjoseph92/distributed that referenced this pull request Oct 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

resumed->rescheduled is an invalid transition

2 participants