Opened 12 years ago

Closed 12 years ago

#19810 closed Bug (fixed)

MemcachedCache should use pickle.HIGHEST_PROTOCOL

Reported by: fernandogrd@… Owned by: Bas Peschier
Component: Core (Cache system) Version: dev
Severity: Normal Keywords: sprint2013
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

MemcachedCache does not specify pickle protocol, and python-memcached defaults to version 0. Since others backends use HIGHEST_PROTOCOL, MemcachedCache should use too, to keep the standard.

Related to #16378 (ticket)

Change History (12)

comment:2 by anonymous, 12 years ago

Has patch: set

comment:3 by Claude Paroz, 12 years ago

Triage Stage: Unreviewed → Accepted

Accepting based on the discussion on #16378.

comment:4 by lvh_, 12 years ago

Needs tests: set

There should probably be a test that checks that it actually uses HIGHEST_PROTOCOL :)

comment:5 by lvh_, 12 years ago

Owner: changed from nobody to lvh_
Status: new → assigned

comment:6 by lvh_, 12 years ago

Owner: lvh_ removed
Status: assigned → new

comment:7 by Aymeric Augustin, 12 years ago

#11011 was a duplicate.

comment:8 by Bas Peschier, 12 years ago

Keywords: sprint2013 added
Owner: set to Bas Peschier
Status: new → assigned

comment:9 by Aymeric Augustin, 12 years ago

It looks like #15815 is the same thing, or at least very closely related.

comment:11 by Bas Peschier, 12 years ago

Needs tests: unset

comment:12 by Aymeric Augustin <aymeric.augustin@…>, 12 years ago

Resolution: → fixed
Status: assigned → closed

In 5a9b2bce242bd2f8a9fed6ac735406ce68b10738:

Fixed #19810 -- MemcachedCache now uses pickle.HIGHEST_PROTOCOL

Note: See TracTickets for help on using tickets.
Back to Top