Skip to content

Loading…

DCOM-32: Memcache cache relies on deprecated functions #628

Closed
doctrinebot opened this Issue · 7 comments

2 participants

@doctrinebot

Jira issue originally created by user sebastian.hoitz:

The method getIds() in MemcacheCache relies on the old "cachedump" stat type.

But as you can read here: http://de2.php.net/manual/en/memcache.getextendedstats.php this has been removed due to security reasons.

@doctrinebot

Comment created by sebastian.hoitz:

Adding this to memcached before getting the extended stats cachedump fixed this issue for me:

if(!is_int($slabId)) {
    continue;
}
@doctrinebot

Comment created by @guilhermeblanco:

This issue doesn't seen to be valid anymore based on commit of @hobodave on Jan 29th.

Please reopen if it is still valid. I could not reproduce.

@doctrinebot

Comment created by @guilhermeblanco:

Memcache daemon 1.4.5 do not provide cachedump and triggers a couple of issues all around.
Here is a link that gives more information: http://www.pecl.php.net/bugs/bug.php?id=20375&edit=3

We need to think on a workaround since current state of Doctrine 2 is unusable with recent memcache.

@doctrinebot

Comment created by @guilhermeblanco:

Fixed in master by this commit: 4861698

@doctrinebot

Issue was closed with resolution "Fixed"

@doctrinebot

Comment created by sobstel:

@Guilherme, your fix introduced big performance issue as now for each fetch() call there are always 2 additional calls, which is kind of big overhead. See #125 for details.

@doctrinebot doctrinebot added the Bug label
@doctrinebot doctrinebot added this to the 2.2 milestone
@doctrinebot doctrinebot closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.