mirror of
https://github.com/hibiken/asynq.git
synced 2026-04-26 07:25:56 +03:00
[GH-ISSUE #529] [BUG] Client side aggregation tasks stay in "aggregating" state forever #2272
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#2272
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?
Originally created by @atkinsonbg on GitHub (Aug 27, 2022).
Original GitHub issue: https://github.com/hibiken/asynq/issues/529
Originally assigned to: @hibiken on GitHub.
Describe the bug
I using client side aggregation and the tasks are making it to the queue, however they stay in "aggregating" state forever. In this case "forever" has been 24+ hours.
To Reproduce
Steps to reproduce the behavior (Code snippets if applicable):
`client := asynq.NewClient(asynq.RedisClientOpt{Addr: redisAddr})
defer client.Close()
Expected behavior
Screenshots
See above
Environment (please complete the following information):
asynqpackage v0.23.0Additional context
Running this in a Docker container with a Redis docker container. If I go into AsynqMon and hit the run button on the tasks they execute successfully. Am I not Enqueueing correctly, or do I need to manually set a grace period?
@atkinsonbg commented on GitHub (Aug 27, 2022):
Now that I've stepped away for a day I think I'm reading the docs wrong: https://github.com/hibiken/asynq/wiki/Task-aggregation
I need to queue them on the client, but I also need to tell the server how to aggregate those tasks? OK, that makes sense. However, is there a pattern on how I can provide multiple aggregates to the server?
@atkinsonbg commented on GitHub (Aug 27, 2022):
Ok, sorry, closing this issue as it’s not an issue. Just needed a day to walk away and come back with fresh eyes. I implemented the aggregator per your example and it works as expected.
Thanks for working on this, I’m really enjoying the functionality!