Skip to content

Fix cancelled running scheduled task state - #14109

Merged
Doc94 merged 2 commits into
PaperMC:mainfrom
metabrixkt:bug/fix-cancelled-scheduled-task-state
Oct 24, 2026
Merged

Doc94 merged 2 commits into
PaperMC:mainfrom
metabrixkt:bug/fix-cancelled-scheduled-task-state

Conversation

@metabrixkt

@metabrixkt metabrixkt commented Jul 26, 2026 •

Copy link
Copy Markdown
Contributor

Fixes #14107

Repeating scheduled tasks cancelled while they are running currently stop being rescheduled, but remain in CANCELLED_RUNNING after their callback finishes.

This finalizes the cancellation in all three Folia-friendly schedulers that have a task state machine once the current execution completes. The task then reports CANCELLED, and later calls to cancel() return CANCELLED_ALREADY.

[edit] And one more thing: Folia is also gonna need a minor change in the region scheduler (beyond updating the upstream commit), which I'll probably PR shortly after this one merges (if it merges quickly)

@metabrixkt
metabrixkt requested a review from a team as a code owner July 26, 2026 18:10
@github-project-automation github-project-automation Bot moved this to Awaiting review in Paper PR Queue Jul 26, 2026
@metabrixkt
metabrixkt force-pushed the bug/fix-cancelled-scheduled-task-state branch from d19b066 to 6131264 Compare August 2, 2026 19:08
@metabrixkt

Copy link
Copy Markdown
Contributor Author

Rebased, no changes

@metabrixkt
metabrixkt force-pushed the bug/fix-cancelled-scheduled-task-state branch from 6131264 to eab537e Compare August 24, 2026 06:29
@metabrixkt

Copy link
Copy Markdown
Contributor Author

Rebased, no changes

@Warriorrrr Warriorrrr added the type: bug Something doesn't work as it was intended to. label Oct 8, 2026
@papermc-projects papermc-projects Bot moved this from Delayed approval to Full approval in Paper PR Queue Oct 23, 2026
@Doc94
Doc94 merged commit 05c2798 into PaperMC:main Oct 24, 2026
4 checks passed
@github-project-automation github-project-automation Bot moved this from Full approval to Merged in Paper PR Queue Oct 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug Something doesn't work as it was intended to.

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

Repeating ScheduledTask stays CANCELLED_RUNNING after its callback finishes

3 participants