Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Memcache support #4481

Closed
drysdaleb opened this Issue May 14, 2013 · 9 comments

Comments

Projects
None yet
7 participants

In ZF1 the cache supports both the memcache extension through Zend_Cache_Backend_Memcached, and the memcached extension (note the 'D') through Zend_Cache_Backend_Libmemcached.

In ZF2 however, there only appears to be support for the memcached extension.

This is a problem when deploying the framework on systems that don't support libmemcached such as Windows.

Is the lack of a Storage Adapter in Zf2 which supports memcache extension an oversight, or is there a reason why it wasn't included? I've tried finding information on in a few places but can't track anything cocrete down.

If this is an oversight, would a proposal to add support for memcache to a future release be considered if I submitted one?

Member

marc-mabe commented May 15, 2013

That's because the lack of time implementing it and because php-memcache seams to be not maintained very well (version 3.* isn't stable since 2007)

Feel free to submit a PR!

I don't know if php-memcached / libmemcached is available for windows - if it's not -> an adapter for php-memcache would be must have feature!

PS: I think the min supported version have to be at least the last stable!

Ok thanks Mark,

I hadn't spotted that the current stable was 2.2.7. In my own experiences
php-memcached isn't available for windows as there's no official supported
build of libmemcached on windows to build against, hence why I asked about
memcache instead.

I think it best to resolve this then through submission of a PR to be
included in a future build of the framework. While php-memcache is limited
feature-wise when compared to php-memcached, 95% of whats required by the
StorageInterface should be possible to do with whats there, other parts may
need to gracefully fail or require a more 'creative' solution.

I think as a minimum the PR should aim to support is the current stable
2.2.7 build of php-memcache also.

Right, off to read up on how to submit a PR so. Should this issue remain
open so, or do you think it best to close it as something to be fixed later?

Brian

On 15 May 2013 16:06, Marc Bennewitz notifications@github.com wrote:

That's because the lack of time implementing it and because php-memcacheseams to be not maintained very well (version 3.* isn't stable since 2007)

Feel free to submit a PR!

I don't know if php-memcached / libmemcached is available for windows -
if it's not -> an adapter for php-memcache would be must have feature!

PS: I think the min supported version have to be at least the last stable!


Reply to this email directly or view it on GitHubhttps://github.com/zendframework/zf2/issues/4481#issuecomment-17944322
.

Member

marc-mabe commented May 15, 2013

@drysdaleb Sure it's possible to implement with a limited set of features.
@weierophinney please mark with the labels enhancemant & Cache

hidahua commented Aug 7, 2013

I also encountered this problem~
I can not find the available file: memcached.dll

Member

Ocramius commented Aug 7, 2013

@hidahua unrelated? That's a problem with your PHP installation.

hidahua commented Sep 2, 2013

I'm in the windows operating system, use zend server, I did not find a suitable memcached dll files

Member

marc-mabe commented Sep 2, 2013

@hidahua As noted above the memcache d extension seems to be not available on windows but memcache does. Feel free to implement a storage adapter for it and send a PR.

Contributor

cgmartin commented Feb 14, 2014

Started work on a Memcache adapter: https://github.com/cgmartin/zf2/tree/memcache-adapter
Trying to keep it relatively similar to the Memcached adapter so that configs won't have to change much if you need to switch php extensions for any reason. Will create a PR once complete.

weierophinney added a commit that referenced this issue Mar 5, 2014

Contributor

jmleroux commented Mar 10, 2014

@weierophinney
This ticket should be closed, as there is a PR for this feature

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment