-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
celery队列替代django-q队列 #2809
base: master
Are you sure you want to change the base?
celery队列替代django-q队列 #2809
Conversation
这个PR确实很大! 我有一个建议, 你分成几步:
这样你可以把这个任务分解成小任务, 我们其他的有需要的有时间的人也可以帮你完成. |
第二步里面, 可以尝试做一些 decorator 或者转换的相关函数, 这样可能后续是可以复用的 |
https://benninger.ca/posts/celery-serializer-pydantic/ 还有类似这样的自动序列化工具, 也有可能是一个可行的方向 |
好的感谢,目前所有的任务调用点,和任务修改成celery这种基本完成了,我手动部署再本地,完成了,基本测试原来所有通知,都已经可以正常用。 2.目前存在有多个worker接受到同一task ,任务重复执行问题 celery/celery#4400 ? |
|
好的,我先尝试解决以下前两个问题,感谢大佬的建议 |
1.可以考虑使用beat 实现 ETA任务 |
这个问题你有解决方法么?可以指导以下,目前我依然在本地出现这个问题 |
4bace95
to
2f0d0f4
Compare
f174b62
to
5579f7a
Compare
e69b56d
to
72f49bd
Compare
@LeoQuote 目前看所有功能都能通过了,这个单元测试需要大佬们帮忙看下 |
因为django-q无法实现多队列,且项目已经很久没更新了。django-q2虽然还在更新,但是依然不支持多队列,和优先级。
之前的pr比较乱,重新提交下,发现改动点好多。。好多单元测试都需要。。。