-
Notifications
You must be signed in to change notification settings - Fork 264
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
fix CelerySignalProcessor delete handling #473
base: master
Are you sure you want to change the base?
Conversation
1. Removed unecessary comma from setter of `bulk_data` variable. 2. Add `doc_module` to `registry_delete_task` function parameters so that we can init the registered document correctly.
@Bragegs After trying your changes I get the following error in DigitalOcean, is there a way to avoid this?:
|
@Teachmetech could this be a DigitalOcean issue? https://stackoverflow.com/a/67347498 https://www.digitalocean.com/community/tutorials/python-multiprocessing-example |
Interesting, thanks for that info! Is it possible to workaround this issue by having the deletes go through |
This version seems to actually delete all indexes and related docs instances correctly
bulk_data
variable.registry_delete_task
function parameter namedata
intobulk_data
doc_module
toregistry_delete_task
function parameters so that we can init the registered document correctly.Fixes 472