Implementation of `temp-file-store` creates too many futures #74

Closed
alexott opened this Issue Aug 12, 2012 · 2 comments

2 participants

@alexott

Implementation of temp-file-store in the ring.middleware.multipart-params.temp-file creates too many futures objects and this leads to too many threads running on web services that handles many multipart uploads...

P.S. I'll try to refactor code to use single expiration thread or something like...

@weavejester
Collaborator

The canonical repository is now at: https://github.com/ring-clojure/ring

Issues should be raised there.

In this case, it looks like the logic for the store is wrong; the expiration thread should only be created once, but it has been placed inside the storage function, not outside it.

@alexott

Oops, I'm sorry - I forgot about ring organization - I will create issue there...

@alexott alexott closed this Aug 12, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment