Re: Patch review #2070

2006-08-18 Thread [EMAIL PROTECTED]
http://code.djangoproject.com/attachment/ticket/2070/3581-streaming_uploads_and_uploadprogress_middleware_code_cleanup_review_buffer_size_setting.diff Added setting to middleware. --~--~-~--~~~---~--~~ You received this message because you are subscribed to the G

Re: Patch review #2070

2006-08-18 Thread [EMAIL PROTECTED]
Streaming uploads is now a middleware, so you can choose to load it or not. But a setting for buffer size is a good idea. --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups "Django developers" group. To post to this g

Re: Patch review #2070

2006-08-18 Thread Ivan Sagalaev
[EMAIL PROTECTED] wrote: > http://code.djangoproject.com/attachment/ticket/2070/3581-streaming_uploads_and_uploadprogress_middleware_code_cleanup_review.diff One thing I disagree with is that streaming on disk is done unconditionally. This is very good for files the size of tens of megabytes bu

Patch review #2070

2006-08-18 Thread [EMAIL PROTECTED]
http://code.djangoproject.com/attachment/ticket/2070/3581-streaming_uploads_and_uploadprogress_middleware_code_cleanup_review.diff --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups "Django developers" group. To post t