Improved memcached module
Maxim Dounin
mdounin at mdounin.ru
Fri Aug 21 14:21:31 MSD 2009
Hello!
On Fri, Aug 21, 2009 at 01:40:37PM +0400, Igor Sysoev wrote:
> On Fri, Aug 21, 2009 at 12:47:24PM +0400, Maxim Dounin wrote:
>
> > Hello!
> >
> > On Fri, Aug 21, 2009 at 10:06:00AM +0200, Spil Games wrote:
> >
> > > Hi,
> > >
> > > Attached is a patch which allows both custom headers and custom response
> > > codes to be stored in memcached and returned by the memcached module.
> > > The Nginx 0.6 version of this patch has been in production on a heavily
> > > loaded site (100M unique visitors per month) for over a year with no
> > > problems whatsoever. The original thread on this can be found here:
> > >
> > > http://www.ruby-forum.com/topic/160379#724717
> > >
> > > I think it would be a valuable addition to Nginx to include this in the
> > > main distribution, but I'll leave this up to Igor. The patch may be a
> > > bit too hacky for his taste.
> > >
> > > Feedback appreciated!
> > >
> > > Attachments:
> > > http://www.ruby-forum.com/attachment/3956/nginx-0.7.61-memcache.final.patch
> >
> > There are at least 2 problems with this patch I see:
> >
> > 1. It relies on 'HTTP/' prefix on to find out if key in question
> > has headers. This is bad, as there is no way to return text
> > starting from this string. IMHO it should be changed to use flags
> > instead.
>
> I agree, it should use a flag and the flag should not conflict with two
> perl Cache-Memcached module's flags:
> F_STORABLE 0x0001
> F_COMPRESS 0x0002
There are a lot more flags for memcached out there as used by
various clients (see http://www.hjp.at/zettel/m/memcached_flags.rxml),
so it probably should be configurable instead.
Maxim Dounin
More information about the nginx
mailing list