Drupal 6 plans? [#188474]

Comment	File	Size	Author
#16	mcd6jv.patch	27.71 KB	jvandyk
#6	memcache_d6.patch	7.95 KB	kirie
#2	memcache.inc_.patch	1.21 KB	Anonymous (not verified)

Comment #1

zarin CreditAttribution: zarin commented 9 March 2008 at 20:17

up

Log in or register to post comments

Comment #2

Anonymous (not verified) CreditAttribution: Anonymous commented 11 March 2008 at 06:43

File	Size
memcache.inc_.patch	1.21 KB

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?

Log in or register to post comments

Comment #3

Anonymous (not verified) CreditAttribution: Anonymous commented 11 March 2008 at 06:56

Status:

Active

» Needs work

setting to code needs work...

Log in or register to post comments

Comment #4

kirie CreditAttribution: kirie commented 27 March 2008 at 05:49

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.

Log in or register to post comments

Comment #5

Anonymous (not verified) CreditAttribution: Anonymous commented 27 March 2008 at 06:21

@kirie: cool, looking forward to it.

Log in or register to post comments

Comment #6

kirie CreditAttribution: kirie commented 30 March 2008 at 07:19

Status:

Needs work

» Needs review

File	Size
memcache_d6.patch	7.95 KB

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?

Log in or register to post comments

Comment #7

slantview CreditAttribution: slantview commented 31 March 2008 at 18:49

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

Log in or register to post comments

Comment #8

dmgorsky CreditAttribution: dmgorsky commented 15 April 2008 at 14:36

Up :)

Log in or register to post comments

Comment #9

GoofyX CreditAttribution: GoofyX commented 7 May 2008 at 20:15

Subscribing.

Log in or register to post comments

Comment #10

triclops CreditAttribution: triclops commented 29 May 2008 at 03:36

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

subscribes

Log in or register to post comments

Comment #11

jcfiala CreditAttribution: jcfiala commented 21 August 2008 at 17:05

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

Log in or register to post comments

Comment #12

Jeremy CreditAttribution: Jeremy commented 10 September 2008 at 21:28

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?

Log in or register to post comments

Comment #13

patchak CreditAttribution: patchak commented 22 September 2008 at 17:00

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

Log in or register to post comments

Comment #14

juan_g CreditAttribution: juan_g commented 12 October 2008 at 00:14

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

Log in or register to post comments

Comment #15

sidharth_k CreditAttribution: sidharth_k commented 13 October 2008 at 16:54

subscribing

Log in or register to post comments

Comment #16

jvandyk CreditAttribution: jvandyk commented 19 October 2008 at 19:51

File	Size
mcd6jv.patch	27.71 KB

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.