According to comments at
http://in3.php.net/manual/en/memcache.delete.php
$success = $mc->delete($full_key);
in function dmemcache_delete($key, $bin = 'cache') {
should be change to:
$success = $mc->delete($full_key, 0);
Without it on my system delete was not working.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | fix_delete-1103796-71dev.patch | 452 bytes | Anonymous (not verified) |
| #4 | 1103796-memcache_delete_fix-D618.patch | 463 bytes | Anonymous (not verified) |
| #4 | 1103796-memcache_delete_fix-D61dev.patch | 463 bytes | Anonymous (not verified) |
| #3 | 1103796-memcache_delete_fix-D618.patch | 471 bytes | Anonymous (not verified) |
| #3 | 1103796-memcache_delete_fix-D61dev.patch | 412 bytes | Anonymous (not verified) |
Comments
Comment #1
drewish commentedThis seems like a dupe of #1101374: Remove warnings and notices. it's got a bunch of unrelated changes. I'm a bigger fan of this patch.
Comment #2
fgm#1101374: Remove warnings and notices. indeed contains 3 changes:
- the 0 argument, like in this one
- a warning removal to work around tiny incompatibilies between some combinations of the memcache extension and memcached daemon
- a bit of refactoring on return value, no functional change
Comment #3
Anonymous (not verified) commentedHere is the patch for 6.x-1.8 and 6.x-1.x-dev versions
Comment #4
Anonymous (not verified) commentedSorry, here are the patches without prefixing.
Comment #5
Anonymous (not verified) commentedWithout this patch, if you run memcached 1.4.5, you'll notice weird behavior with CCK. When you try to update content, the database will have the new value in it, but fields will display old content. To your users it looks like you can't edit/change content. What is happening is that memcache can't delete it's cache, so it continues to serve up stale content.
Comment #6
catchCommitted to 6.x-1.x, moving to D7.
Comment #7
Anonymous (not verified) commentedHere is the patch ported to 7.x-1.x-dev
Comment #8
catchWill review later. Thanks for the re-roll.
Comment #9
catchCommitted to 7.x-1.x, thanks!