I have set in settings.php prefix for site:

 $conf['cache_inc'] = './sites/all/modules/cacherouter/cacherouter.inc';
$conf['cacherouter'] = array(
  'default' => array(
    'engine' => 'apc',
    'prefix' => 'gr_',
  ),
);

And when I see manually in APC cache, I see than prefix don't adds to cache labels. Does apc type of caching support prefixes? Maybe I must see prefixes for cache items in other place?

CommentFileSizeAuthor
#3 cacherouter-5.x-1.0-beta4_patched_map-checking_apc-prefix_3.tar_.gz16.44 KBmurz
#3 cacherouter-5.x-1.0-beta4_apc_prefix_3.patch3.22 KBmurz
#2 cacherouter-5.x-1.0-beta4_apc_prefix_2.patch3.22 KBmurz
#2 cacherouter-5.x-1.0-beta4_patched_map-checking_apc-prefix_2.tar_.gz16.43 KBmurz
#1 cacherouter-5.x-1.0-beta4_apc-prefix.patch1.8 KBmurz
#1 cacherouter-5.x-1.0-beta4_patched_map-checking_apc-prefix.tar_.gz16.4 KBmurz

Comments

murz’s picture

murz
Primary language English
Location Yerevan, Armenia
commented
Status: Active » Needs review
StatusFileSize
new cacherouter-5.x-1.0-beta4_patched_map-checking_apc-prefix.tar_.gz16.4 KB
new cacherouter-5.x-1.0-beta4_apc-prefix.patch1.8 KB

I have fix this issue, patch and patched module (with map checking patch already applied) is attached. I think that files for other engines need to patching too...
It's worked for me good, waiting for other reviews.

murz’s picture

murz
Primary language English
Location Yerevan, Armenia
commented
StatusFileSize
new cacherouter-5.x-1.0-beta4_patched_map-checking_apc-prefix_2.tar_.gz16.43 KB
new cacherouter-5.x-1.0-beta4_apc_prefix_2.patch3.22 KB

Sorry, first patch decrease the lookup_% cache items perfomance, I fix this in second version of patch.

murz’s picture

murz
Primary language English
Location Yerevan, Armenia
commented
StatusFileSize
new cacherouter-5.x-1.0-beta4_apc_prefix_3.patch3.22 KB
new cacherouter-5.x-1.0-beta4_patched_map-checking_apc-prefix_3.tar_.gz16.44 KB

In 3'th version I move prefix for 'lookup_' records to start of string:
Names like lookup_<prefix>-cache_taxonomy changes to <prefix>-lookup_cache_taxonomy

slantview’s picture

slantview commented
Status: Needs review » Fixed

Fixed in latest CVS commit for 5. I backported the way I was doing it in the DRUPAL-6--1 branch. So now we do all prefixing in the Cache class and just use $this->lookup and $this->lock for the correctly prefixed lookup and lock variables and fixed the Cache::key() to use the prefix as well.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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