Thanks Piotr!

One of my developers did the patch manually and she might have introduced
some bugs during the process.

Would it be easier to make it a standalone module rather than a patch for
users?

I'm waiting for your new patch to the latest nginx and I'll test it out and
report back.

Regards,

Bing


--------------------------------------------------
From: "Piotr Sikora" <piotr.sikora@frickle.com>
Sent: Tuesday, November 17, 2009 1:09 PM
To: <nginx@sysoev.ru>
Subject: Re: cache purge

>> I'm investigating cache purge in nginx and currently we have some success
>> with this patch: http://labs.frickle.com/nginx_cache_purge/. The reason
>> I said "some" is the patch works normally but blows out (the message was
>> like "the file to purge is too small...")when the nginx process has been
>> restarted and the cache to purge was created by the previous nginx
>> process.
>
> It's a shame that you didn't report it, because I was unaware of this
> until now.
>
> Anyway, I'm unable to reproduce this bug (purging files from cache after
> restarting nginx works as expected, at least with nginx-0.8.8, so maybe
> you introduced some bugs while syncing it on your own?)
>
> Also, the message you are referring to ("cache file \"%s\" is too small")
> is produced by nginx when it's trying to open cached file (which is also
> done before purging it), so it looks like something is wrong... It would
> be great if you could produce debug log and send it either on- or
> off-list.
>
>> The patch is also a little out of of sync with the latest nginx.
>
> Actually, I was going to sync it with the latest nginx and push the
> release yesterday, together with ngx_supervisord-1.1, but something came
> up and I didn't do it...
>
> But I'm gonna do it in next few days (maybe even today), stay tuned! ;)
>
> Best regards,
> Piotr Sikora < piotr.sikora@frickle.com >
>
>
>