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