celery / django-celery-results

Celery result back end with django
Other
668 stars 206 forks source link

Ensure `delete_expired` uses writable database. #358

Open jacklinke opened 1 year ago

jacklinke commented 1 year ago

Update delete_expired method of ResultManager to use writable database.

Resolves #357

matmunn commented 4 months ago

@jacklinke any joy on this one? I'm running into the same issue that you are

AllexVeldman commented 2 months ago

Just as a heads-up, #427 also needs a testcase with a read-only connection (See my comment for a suggested approach: https://github.com/celery/django-celery-results/pull/427#discussion_r1580787592)