Doctrine Common
  1. Doctrine Common
  2. DCOM-32

Memcache cache relies on deprecated functions

    Details

    • Type: Bug Bug
    • Status: 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

        Sebastian Hoitz created issue -
        Hide
        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
        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
        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 ]
        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
        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
        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[0:20].revisions[0:29],reviews}, methodType=GET}] : Received status code 503 (Service Temporarily Unavailable)

          People

          • Assignee:
            Guilherme Blanco
            Reporter:
            Sebastian Hoitz
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: