mirror of
https://github.com/hibiken/asynq.git
synced 2026-04-26 07:25:56 +03:00
[PR #988] Allow cancel without retry #2982
Labels
No labels
CLI
bug
designing
documentation
duplicate
enhancement
good first issue
good first issue
help wanted
idea
invalid
investigate
needs-more-info
performance
pr-welcome
pull-request
question
wontfix
work in progress
work in progress
work-around-available
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/asynq#2982
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/hibiken/asynq/pull/988
Author: @thedmdim
Created: 12/16/2024
Status: 🔄 Open
Base:
master← Head:master📝 Commits (1)
35edb02allow cancel without retry📊 Changes
1 file changed (+12 additions, -9 deletions)
View changed files
📝
processor.go(+12 -9)📄 Description
Previously we couldn't cancel task without sending it to retry queue. Despite the fact that handler returned
SkipRetryerror, it didn't work correctly. Read from<-ctx.Done()inprocessor.go:246caused by canelation were and will be always earlier than returnedSkipRetryfrom handler. That's why task always went to retry queue.With these changes you are able to cancel task so it will be archived instead of going to retry :)
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.