From 5d7632d3c08e218b72362568e8ba78b82192ed12 Mon Sep 17 00:00:00 2001 From: Alejandro Cabrera Date: Mon, 2 Dec 2013 13:07:45 -0500 Subject: [PATCH] fix: cache contract for memcached backend set() should return a boolean. Currently, it always returns None. This one-line fix resolves this issue. Change-Id: I9d2827891dc8ee8904f3031ea75e5bef0ce30488 --- marconi/common/cache/_backends/memcached.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/marconi/common/cache/_backends/memcached.py b/marconi/common/cache/_backends/memcached.py index c0c788f11..bf0ce9a4b 100644 --- a/marconi/common/cache/_backends/memcached.py +++ b/marconi/common/cache/_backends/memcached.py @@ -57,7 +57,7 @@ class MemcachedBackend(backends.BaseCache): def set(self, key, value, ttl=0): key = self._prepare_key(key) - self._cache.set(key, value, self._get_ttl(ttl)) + return self._cache.set(key, value, self._get_ttl(ttl)) def unset(self, key): self._cache.delete(self._prepare_key(key))