[drupal-devel] [feature] Improved caching with eAccelerator/turck mmcache

[Dries]

Issue status update for http://drupal.org/node/24260

 Project:      Drupal
 Version:      4.6.0
 Component:    base system
 Category:     feature requests
 Priority:     normal
 Assigned to:  Anonymous
 Reported by:  Yrlec
 Updated by:   Dries
 Status:       patch

- The variable names are not consistent with our coding style (no camel
casing).


- The code won't work against HEAD.


- This needs to be benchmarked before it can be committed.  I don't
think it will buy us much.


- I agree with James that the squid changes might be worth looking at
too (forgot the URL).




Dries



Previous comments:
------------------------------------------------------------------------

June 3, 2005 - 21:52 : Yrlec

Attachment: http://drupal.org/files/issues/newcache.php (3.82 KB)

I'm not sure if this is the best place to post, please let me know if I
should post somewhere else.


I've done some changes to bootstrap.inc to make it use the semaphore
cache of either eAccelerator or Turck mmCache instead of the database
to handle the caching. I've not done any benchmarking but I'm quiet
sure that it will speed things up. I'm afraid that I didn't base my
code on the latest version so I apologize if my changes aren't
compatible with the current cvs-version. As you can see, if neither
eAccelerator nor Turck mmCache isn't available on the system the old
caching mechanism will be used.


The changes are in cache_get, cache_set and cache_clear_all. I'm sorry
I didn't post a diff but I think it will be quiet easy to implement
anyway.




------------------------------------------------------------------------

June 3, 2005 - 22:31 : walkah

+1 in principle... however, rather than see this get committed directly,
I'd rather see a "caching framework" evolve...


we've also had other talks about making drupal more squid friendly, in
addition to using memcached, etc.


i'm willing to help put some brainpower into this...