[Django] #29887: support for pymemcache to replace python-memcached
Django
------------------------------------------------+------------------------
Reporter: Adrian Turjak | Owner: nobody
Type: Cleanup/optimization | Status: new
Component: Uncategorized | Version: master
Severity: Normal | Keywords:
Triage Stage: Unreviewed | Has patch: 0
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
------------------------------------------------+------------------------
With python-memcached no longer getting any true support[1], maybe it's
time django adds support for pymemcache[2]?
[1]: https://github.com/linsomniac/python-memcached/issues/95
[2]: https://github.com/pinterest/pymemcache
--
Ticket URL: <https://code.djangoproject.com/ticket/29887>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
Django
Reporter: Adrian Turjak | Owner: nobody
Cleanup/optimization |
Component: Uncategorized | Version: master
Severity: Normal | Resolution:
Keywords: | Triage Stage:
| Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Description changed by Adrian Turjak:
Old description:
> With python-memcached no longer getting any true support[1], maybe it's
> time django adds support for pymemcache[2]?
>
> [1]: https://github.com/linsomniac/python-memcached/issues/95
> [2]: https://github.com/pinterest/pymemcache
New description:
With python-memcached no longer getting any true support(1), maybe it's
time django adds support for pymemcache(2)?
1: https://github.com/linsomniac/python-memcached/issues/95
2: https://github.com/pinterest/pymemcache
--
--
Ticket URL: <https://code.djangoproject.com/ticket/29887#comment:1>
Django
-------------------------------------+-------------------------------------
Reporter: Adrian Turjak | Owner: nobody
Cleanup/optimization |
Component: Core (Cache system) | Version: master
Severity: Normal | Resolution: needsinfo
Keywords: | Triage Stage:
| Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Tim Graham):
* status: new => closed
* component: Uncategorized => Core (Cache system)
* resolution: => needsinfo
Comment:
You should make this proposal on the DevelopersMailingList. If there's
consensus, we'll reopen the ticket.
--
Ticket URL: <https://code.djangoproject.com/ticket/29887#comment:2>
Django
-------------------------------------+-------------------------------------
Reporter: Adrian Turjak | Owner: nobody
Cleanup/optimization |
Component: Core (Cache system) | Version: master
Severity: Normal | Resolution: needsinfo
| Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Comment (by Adrian Turjak):
Mailing list thread started here:
https://groups.google.com/forum/#!topic/django-developers/PAiLQybcGT8
--
Ticket URL: <https://code.djangoproject.com/ticket/29887#comment:3>
Django
Reporter: Adrian Turjak | Owner: nobody
Type: Cleanup/optimization | Status: new
Severity: Normal | Resolution:
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Tim Graham):
* status: closed => new
* resolution: needsinfo =>
* stage: Unreviewed => Accepted
Comment:
The initial response on the mailing list is positive.
--
Ticket URL: <https://code.djangoproject.com/ticket/29887#comment:4>
Django
Reporter: Adrian Turjak | Owner: mainanick
Type: | Status: assigned
Cleanup/optimization |
Component: Core (Cache system) | Version: master
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Changes (by mainanick):
* owner: nobody => mainanick
* status: new => assigned
--
Ticket URL: <https://code.djangoproject.com/ticket/29887#comment:5>
Django
-------------------------------------+-------------------------------------
Type: | Nick
Cleanup/optimization | Status: assigned
Component: Core (Cache system) | Version: master
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Comment (by Kosei Kitahara):
I'm a maintener of [https://github.com/django-pymemcache/django-
pymemcache]
Some API are different from python-memcache and pymemcache.
So I override pymemcache.HashClient and override some method to support
python-memcached's API
--
Ticket URL: <https://code.djangoproject.com/ticket/29887#comment:6>
Django
-------------------------------------+-------------------------------------
Reporter: Adrian Turjak | Owner: Maina
Type: | Nick
Cleanup/optimization | Status: assigned
Component: Core (Cache system) | Version: master
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Comment (by Adrian Turjak):
@Maina Nick
Any progress on this? At the very least marking the backend as deprecated
because the library itself is?
@Kosei Kitahara
That is always good to know, but I think ultimately if we supported this
we're best writing a new standalone backend so that it uses the client
natively and isn't reliant on the other backend which we should deprecate.
--
Ticket URL: <https://code.djangoproject.com/ticket/29887#comment:7>
Django
Reporter: Adrian Turjak | Owner: (none)
Type: Cleanup/optimization | Status: new
Component: Core (Cache system) | Version: master
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Claude Paroz):
* owner: Maina Nick => (none)
* status: assigned => new
--
Ticket URL: <https://code.djangoproject.com/ticket/29887#comment:8>
Django
Reporter: Adrian Turjak | Owner: Nick Pope
Type: New feature | Status: assigned
Component: Core (Cache system) | Version: master
Severity: Normal | Resolution:
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Nick Pope):
* keywords: => pymemcache
* owner: (none) => Nick Pope
* has_patch: 0 => 1
* status: new => assigned
* type: Cleanup/optimization => New feature
Comment:
[https://github.com/django/django/pull/13310 PR]
--
Ticket URL: <https://code.djangoproject.com/ticket/29887#comment:9>
Django
-------------------------------------+-------------------------------------
Reporter: Adrian Turjak | Owner: Nick Pope
Type: New feature | Status: assigned
Component: Core (Cache system) | Version: master
Severity: Normal | Resolution:
Keywords: pymemcache | Triage Stage: Accepted
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Comment (by Mariusz Felisiak <felisiak.mariusz@…>):
In [changeset:"e2013b260a8f689eee280d00b96a9507895ec236" e2013b26]:
{{{
#!CommitTicketReference repository=""
revision="e2013b260a8f689eee280d00b96a9507895ec236"
Refs #29887, #27480 -- Moved touch() to BaseMemcachedCache.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/29887#comment:10>
Django
-------------------------------------+-------------------------------------
Reporter: Adrian Turjak | Owner: Nick Pope
Type: New feature | Status: assigned
Component: Core (Cache system) | Version: master
Severity: Normal | Resolution:
Keywords: pymemcache | Triage Stage: Accepted
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Comment (by Mariusz Felisiak <felisiak.mariusz@…>):
In [changeset:"cc1f2c6a199ca2318acdc59bf0731d2cdd7a4e4a" cc1f2c6a]:
{{{
#!CommitTicketReference repository=""
revision="cc1f2c6a199ca2318acdc59bf0731d2cdd7a4e4a"
Refs #29887 -- Simplified memcached client instantiation.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/29887#comment:11>
Django
-------------------------------------+-------------------------------------
Reporter: Adrian Turjak | Owner: Nick Pope
Type: New feature | Status: assigned
Component: Core (Cache system) | Version: master
Severity: Normal | Resolution:
Keywords: pymemcache | Triage Stage: Accepted
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Comment (by Mariusz Felisiak <felisiak.mariusz@…>):
In [changeset:"a6291394256aa758d74eec9ce0cfae8aea6475f2" a629139]:
{{{
#!CommitTicketReference repository=""
revision="a6291394256aa758d74eec9ce0cfae8aea6475f2"
Refs #29887, Refs #24212 -- Added servers configuration hook for memcached
backends.
The servers property can be overridden to allow memcached backends to
alter the server configuration prior to it being passed to instantiate
the client. This allows avoidance of documentation for per-backend
differences, e.g. stripping the 'unix:' prefix for pylibmc.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/29887#comment:12>
Django
-------------------------------------+-------------------------------------
Reporter: Adrian Turjak | Owner: Nick Pope
Type: New feature | Status: assigned
Component: Core (Cache system) | Version: master
Severity: Normal | Resolution:
Keywords: pymemcache | Triage Stage: Accepted
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Comment (by Mariusz Felisiak <felisiak.mariusz@…>):
In [changeset:"cda0a3d7773b44c28669dfa1a41cb6203c5c298e" cda0a3d7]:
{{{
#!CommitTicketReference repository=""
revision="cda0a3d7773b44c28669dfa1a41cb6203c5c298e"
Refs #29887, Refs #24212 -- Added more client servers tests for
PyLibMCCache.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/29887#comment:13>
Django
-------------------------------------+-------------------------------------
Reporter: Adrian Turjak | Owner: Nick Pope
Component: Core (Cache system) | Version: master
Keywords: pymemcache | Triage Stage: Accepted
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Mariusz Felisiak <felisiak.mariusz@…>):
* status: assigned => closed
* resolution: => fixed
Comment:
In [changeset:"b4d46df5cad6c936d83dd4f8038d0dc1121bc21e" b4d46df5]:
{{{
#!CommitTicketReference repository=""
revision="b4d46df5cad6c936d83dd4f8038d0dc1121bc21e"
Fixed #29887 -- Added a cache backend for pymemcache.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/29887#comment:14>