Skip to content
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

django slow log no longer offloads to celery #1

Open
nijotz opened this issue Apr 17, 2012 · 0 comments
Open

django slow log no longer offloads to celery #1

nijotz opened this issue Apr 17, 2012 · 0 comments

Comments

@nijotz
Copy link

nijotz commented Apr 17, 2012

django slow log was modified to log the record on request and update it on response. In doing so, it no longer offloads to celery, even if configured to do so. It would be nice to keep the "initial save on request, update on response" feature and to be able to offload both database transactions to celery.

It seems like the offload configuration should be updated to support specifying which database transaction to offload: the pre-request, the pre-response, both or none.

Race conditions between the two offloaded tasks would need to be avoided. The pre-response could be passed the celery task ID of the pre-request and make sure it has completed before attempting to update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant