← Back to team overview

launchpad-reviewers team mailing list archive

Re: [Merge] ~jugmac00/launchpad:replace-python-memcached-with-pymemcache into launchpad:master

 

`MemcachedLayer.setUp` deliberately calls `memcache_client_factory` and attempts to set a test value in it before it starts `memcached`, in order to check whether a `memcached` process is already running before starting a new one.  The old library returned 0 from its `set` method in this case, but the new one raises an exception.

https://paste.ubuntu.com/p/gy2dgPxYkD/ causes it to get further.  There are still various test failures, but hopefully they should be easier to investigate at this point.
-- 
https://code.launchpad.net/~jugmac00/launchpad/+git/launchpad/+merge/411523
Your team Launchpad code reviewers is requested to review the proposed merge of ~jugmac00/launchpad:replace-python-memcached-with-pymemcache into launchpad:master.



Follow ups

References