Project:Memcache API and Integration
Version:5.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

Is there time plans for update project to upcoming Drupal 6 API? Thanks.

Comments

#1

up

#2

lightly tested patch attached which changes the function signature of cache_set in memcache.inc as per the changes from cache_set in drupal 5 to cache_set in drupal 6.

i'm interested in doing some real work here - has anyone else already started on this?

AttachmentSize
memcache.inc_.patch 1.21 KB

#3

Status:active» needs work

setting to code needs work...

#4

We have a couple of guys looking into making a basic workable package for D6. We'll update this thread with files/comments when there are any updates - we'll hopefully have something over the weekend.

#5

@kirie: cool, looking forward to it.

#6

Status:needs work» needs review

This patch (rolled against HEAD) updates both memcache.inc and memcache.db.inc. There is not a lot of change here, the biggest being the cache_get and cache_get function signatures. I've also changed a parameter name ($key to $cid), a variable name ($time to $created) and some function documentation to bring it more in line with D6. Also added in cache_clear_all(NULL, 'cache_block'); as per D6.

The total number of required files to make memcache work correctly in D6 have been drastically reduced, as there is no need to patch core or update tables:
dmemcache.inc
memcache.db.inc
memcache.inc

As there is not need to use the existing memcache.module (actually its .install file) to update tables, I think the functionality in memcache_admin could be moved here?

AttachmentSize
memcache_d6.patch 7.95 KB

#7

Status:needs review» needs work

I will have a D6 branch later this week, worse case, end of next week. Thanks for submitting this, but I will be combining some of the work I already put into Cache Router - http://drupal.org/project/cacherouter

#8

Up :)

#9

Subscribing.

#10

woops. why didn't I see this before I did this... http://drupal.org/node/264015

subscribes

#11

Any updates on the Drupal 6 port? I need this for a project, so I'm willing to donate some time.

#12

Priority:minor» normal

What's the status on this porting effort? How can we best pitch in? @slantview, you suggest you've got your own changes you want to integrate rather than the posted patches, any luck with that?

#13

Will there be any d6 version of this project anytime soon? I'm waiting for this to upgrade some sites, since the patch has been posted so long ago would be nice to revisit this issue and keep it up to date?

Thanks a lot for the work,
Patchak

#14

A Drupal 6 possibility for those in a hurry is the memcache option of the Cache Router mentioned above by slantview.

#15

subscribing

#16

This patch is against the 6--1 branch (though there is no 6.x-1.x-dev option in the Version dropdown, so I'm leaving it as 5.x-1.x-dev). It is based on kirie's changes in #6 but also ports the memcache_admin module to Drupal 6 so that the state of memcache bins can be inspected.

AttachmentSize
mcd6jv.patch 27.71 KB

#17

Status:needs work» fixed

6.x-1.0 release is out, so closing this issue.

#18

Status:fixed» closed (fixed)

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