Re: [wp-svn] [3749] trunk/wp-admin: Remove those naughty flushes.

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: [wp-svn] [3749] trunk/wp-admin: Remove those naughty flushes.

Mike Little
On 4/22/06, [hidden email] <[hidden email]> wrote:
>
>  Revision 3749 Author ryan Date 2006-04-22 02:10:42 +0000 (Sat, 22 Apr 2006)
> Log Message Remove those naughty flushes.
>

Why?
Don't they prevent timeouts (and give valuable feedback) on really
large imports?

Mike
--
Mike Little
http://zed1.com/
_______________________________________________
wp-hackers mailing list
[hidden email]
http://lists.automattic.com/mailman/listinfo/wp-hackers
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: Re: [wp-svn] [3749] trunk/wp-admin: Remove those naughty flushes.

Ryan Boren
Mike Little wrote:

> On 4/22/06, [hidden email] <[hidden email]> wrote:
>
>> Revision 3749 Author ryan Date 2006-04-22 02:10:42 +0000 (Sat, 22 Apr 2006)
>>Log Message Remove those naughty flushes.
>>
>
>
> Why?
> Don't they prevent timeouts (and give valuable feedback) on really
> large imports?

They were screwing up plugins that manipulated the output buffer,
leading lots of bug reports.  There's probably a better way to deal with
it, but I just removed them for now.

Ryan
_______________________________________________
wp-hackers mailing list
[hidden email]
http://lists.automattic.com/mailman/listinfo/wp-hackers
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: Re: [wp-svn] [3749] trunk/wp-admin: Remove those naughty flushes.

Mike Little
On 4/22/06, Ryan Boren <[hidden email]> wrote:

> Mike Little wrote:
> > Why?
> > Don't they prevent timeouts (and give valuable feedback) on really
> > large imports?
>
> They were screwing up plugins that manipulated the output buffer,
> leading lots of bug reports.  There's probably a better way to deal with
> it, but I just removed them for now.
>
> Ryan

Ah right! OK, that makes sense.

Which reminds me I found, and fixed, an MT import bug. I'll raise it
on trac and prepare a patch.


Mike
--
Mike Little
http://zed1.com/
_______________________________________________
wp-hackers mailing list
[hidden email]
http://lists.automattic.com/mailman/listinfo/wp-hackers
Loading...