#2070: [patch] Large streaming uploads
----------------------------------------+-----------------------------------
Reporter: [EMAIL PROTECTED] | Owner: jacob
Status: assigned | Component: Core framework
Version: | Resolution:
Keywords: streaming, upload, large | Stage: Accepted
Has_patch: 1 | Needs_docs: 1
Needs_tests: 0 | Needs_better_patch: 1
----------------------------------------+-----------------------------------
Comment (by Ivan Sagalaev <[EMAIL PROTECTED]>):
A couple of comments from quick reading of 4459-streaming-file-upload.diff
* It streams uploads to files only if FILE_UPLOAD_DIR is set. But it
looks that it stores files in temp directory anyway. What's the point of
FILE_UPLOAD_DIR then?
* Instead of FILE_UPLOAD_DIR it would be nice to have a setting like
MAX_FILE_UPLOAD_BUFFER with the maximum number of bytes stored in memory
during an upload. And only if the file is bigger it is streamed to disc.
This will have a nice effect of not touching disc when dealing with small
files which is faster
* It looks like any Exception during the parsing is swallowed and
returned as a dict of errors that is then stored in a POST data. This
actually breaks the whole exceptions paradigm :-).
* All the save_FIELD_file now accept raw_field. This is backward-
incompatible but worse it limits this method to work only with uploaded
files while now save_FIELD_file can accept any content not matter how it
was created in a program. I understand that just renaming streamed upload
is a fast feature but I believe it deserves a new method. Like
'save_FIELD_from_file' or something...
--
Ticket URL: <http://code.djangoproject.com/ticket/2070#comment:98>
Django Code <http://code.djangoproject.com/>
The web framework for perfectionists with deadlines
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---
