Doctrine Common

Memcache cache relies on deprecated functions

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.0.0-RC2
  • Fix Version/s: 2.2
  • Component/s: Caching
  • Labels:
    None

Description

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.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Sebastian Hoitz added a comment - - edited

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

if(!is_int($slabId)) {
    continue;
}
Show
Sebastian Hoitz added a comment - - edited Adding this to memcached before getting the extended stats cachedump fixed this issue for me: if (!is_int($slabId)) { continue ; }
Hide
Permalink
Guilherme Blanco added a comment -

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.

Show
Guilherme Blanco added a comment - 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.
Guilherme Blanco made changes -
Field Original Value New Value
Status Open [ 1 ] Closed [ 6 ]
Resolution Invalid [ 6 ]
Hide
Permalink
Guilherme Blanco added a comment -

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.

Show
Guilherme Blanco added a comment - 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.
Guilherme Blanco made changes -
Resolution Invalid [ 6 ]
Status Closed [ 6 ] Reopened [ 4 ]
Assignee Roman S. Borschel [ romanb ] Guilherme Blanco [ guilhermeblanco ]
Hide
Permalink
Denis added a comment -

There additionally seems to be a hard-coded limit to the size of the dump:

http://stackoverflow.com/questions/4363904/is-there-any-length-limitation-of-result-by-stats-cachedump-in-memcached

http://lists.danga.com/pipermail/memcached/2007-April/003906.html

Show
Denis added a comment - There additionally seems to be a hard-coded limit to the size of the dump: http://stackoverflow.com/questions/4363904/is-there-any-length-limitation-of-result-by-stats-cachedump-in-memcached http://lists.danga.com/pipermail/memcached/2007-April/003906.html
Hide
Permalink
Guilherme Blanco added a comment -

Fixed in master by this commit: https://github.com/doctrine/common/commit/486169851ea87b3e14ed45d5bfd7d07b1d41af65

Show
Guilherme Blanco added a comment - Fixed in master by this commit: https://github.com/doctrine/common/commit/486169851ea87b3e14ed45d5bfd7d07b1d41af65
Guilherme Blanco made changes -
Status Reopened [ 4 ] Resolved [ 5 ]
Fix Version/s 3.0.x [ 10152 ]
Resolution Fixed [ 1 ]
Hide
Permalink
Przemek Sobstel added a comment -

@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 https://github.com/doctrine/common/pull/125 for details.

Show
Przemek Sobstel added a comment - @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 https://github.com/doctrine/common/pull/125 for details.
Benjamin Eberlei made changes -
Workflow jira [ 12243 ] jira-feedback2 [ 17883 ]
Benjamin Eberlei made changes -
Workflow jira-feedback2 [ 17883 ] jira-feedback3 [ 19846 ]
This list may be incomplete, as errors occurred whilst retrieving source from linked applications:
  • Request to http://www.doctrine-project.org/fisheye/ failed: Error in remote call to 'FishEye 0 (http://www.doctrine-project.org/fisheye/)' (http://www.doctrine-project.org/fisheye) [AbstractRestCommand{path='/rest-service-fe/search-v1/crossRepositoryQuery', params={query=DCOM-32, expand=changesets[-21:-1].revisions[0:29],reviews}, methodType=GET}] : Received status code 503 (Service Temporarily Unavailable)

People

  • Assignee:
    Guilherme Blanco
    Reporter:
    Sebastian Hoitz
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: