Skip to content

Commit d704c30

Browse files
jjorissen52auvipy
authored andcommitted
update behavior of app.result_expires to match celery docs
Celery documentation indicates that app.result_expires of CELERY_RESULT_EXPIRES should respect values of 0 or None: "A value of None or 0 means results will never expire (depending on backend specifications)."
1 parent f587855 commit d704c30

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

django_celery_results/backends/database.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -198,6 +198,8 @@ def _forget(self, task_id):
198198

199199
def cleanup(self):
200200
"""Delete expired metadata."""
201+
if not self.expires:
202+
return
201203
self.TaskModel._default_manager.delete_expired(self.expires)
202204
self.GroupModel._default_manager.delete_expired(self.expires)
203205

0 commit comments

Comments
 (0)