Enable Celery Background Manager for additional Backends - #3733
Enable Celery Background Manager for additional Backends#3733datenzauberai wants to merge 10 commits into
Conversation
|
Thanks for creating this PR! Is it in a state where we can review it? |
I'm on it, but this still needs work! The manager itself works locally, but I want to have good tests first so that this can be maintained in the future. |
|
|
@T4rk1n Maybe you could have a quick look. This should work now with the filesystem backend in Celery. Enabling the other backends would need a more substantial rework, because we would need to use get_result/store_result instead of get/set/delete/expire in BaseKeyValueStoreBackend. I wonder if it's really worth the effort to rework this, because I guess that many simple cases can now be handled using WebSocket Callbacks. What do you think? |



This is an implementation to fix #3729 and #3116.
Contributor Checklist
BaseKeyValueStoreBackendBaseKeyValueStoreBackendget_resultandstore_resultoptionals
CHANGELOG.md