We noticed an abnormally high number of hits on CacheRouter::__init while profiling our site running under memcache (i.e. 78 instead of an expected 4 - 6).

Upon investigation noticed many references to $map in CacheRouter instead of $this->map. This has the effect of creating multiple of instances of $bin each time CacheRouter::get or CacheRouter::set are called - this probably works against static caching, etc.

Patch fixes this issue. Made against HEAD.

This appears to be have been fixed in the large patch in http://drupal.org/node/262830 also.

This patch was developed by IDG Australia

CommentFileSizeAuthor
cacherouter.map_.patch1.77 KBtriclops

Comments

slantview’s picture

Status: Needs review » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.