Memcached: Remove retry extra logging and feature flag

This commit is contained in:
Robert Ditthardt
2014-11-18 15:38:52 -08:00
committed by Neil Williams
parent 1d4bf69bd9
commit 6ad89f49b2

View File

@@ -39,7 +39,6 @@ from r2.lib.hardcachebackend import HardCacheBackend
from r2.lib.sgm import sgm # get this into our namespace so that it's
# importable from us
from r2.config import feature
# This is for use in the health controller
_CACHE_SERVERS = set()
@@ -101,24 +100,20 @@ class CMemcache(CacheUtils):
def retry(self, times, fn):
if feature.is_enabled('memcached_retry'):
ex = None
for i in xrange(times):
try:
return fn()
except (pylibmc.NotFound,
pylibmc.BadKeyProvided,
pylibmc.UnknownStatKey,
pylibmc.InvalidHostProtocolError,
pylibmc.NotSupportedError):
raise
except MemcachedError as e:
ex = e
g.log.info('Memcached error, retrying: %r', e)
ex = None
for i in xrange(times):
try:
return fn()
except (pylibmc.NotFound,
pylibmc.BadKeyProvided,
pylibmc.UnknownStatKey,
pylibmc.InvalidHostProtocolError,
pylibmc.NotSupportedError):
raise
except MemcachedError as e:
ex = e
raise MemcachedMaximumRetryException(ex)
else:
return fn()
raise MemcachedMaximumRetryException(ex)
def get(self, key, default = None):
def do_get():