Merge pull request #19 from AveryLevin/add_logging

Add Exception Logging

Author: KFoxder

cache_helper/__init__.py

@@ -1 +1 @@
-VERSION = (1, 0, 3)
+VERSION = (1, 0, 4)

cache_helper/decorators.py

@@ -1,3 +1,5 @@
+import logging
+
 try:
     from _pylibmc import Error as CacheSetError
 except ImportError:
@@ -10,6 +12,7 @@
 
 from cache_helper import utils
 
+logger = logging.getLogger(__name__)
 
 def cached(timeout):
     def _cached(func):
@@ -23,6 +26,10 @@ def wrapper(*args, **kwargs):
             try:
                 value = cache.get(cache_key)
             except Exception:
+                logger.warning(
+                    f'Error retrieving value from Cache for Key: {function_cache_key}',
+                    exc_info=True,
+                )
                 value = None
 
             if value is None:
@@ -34,7 +41,10 @@ def wrapper(*args, **kwargs):
                     cache.set(cache_key, value, timeout)
 
                 except CacheSetError:
-                    pass
+                    logger.warning(
+                        f'Error saving value to Cache for Key: {function_cache_key}',
+                        exc_info=True,
+                    )
 
             return value
 
@@ -71,6 +81,10 @@ def wrapper(*args, **kwargs):
             try:
                 value = cache.get(cache_key)
             except Exception:
+                logger.warning(
+                    f'Error retrieving value from Cache for Key: {function_cache_key}',
+                    exc_info=True,
+                )
                 value = None
 
             if value is None:
@@ -81,7 +95,10 @@ def wrapper(*args, **kwargs):
                 try:
                     cache.set(cache_key, value, timeout)
                 except CacheSetError:
-                    pass
+                    logger.warning(
+                        f'Error saving value to Cache for Key: {function_cache_key}',
+                        exc_info=True,
+                    )
 
             return value
 
@@ -132,10 +149,14 @@ def __get__(self, obj, objtype):
             return fn
 
         def __call__(self, *args, **kwargs):
-            cache_key = self.create_cache_key(*args, **kwargs)
+            cache_key, function_cache_key = self.create_cache_key(*args, **kwargs)
             try:
                 value = cache.get(cache_key)
             except Exception:
+                logger.warning(
+                    f'Error retrieving value from Cache for Key: {function_cache_key}',
+                    exc_info=True,
+                )
                 value = None
             if value is None:
                 value = self.func(*args, **kwargs)
@@ -145,7 +166,10 @@ def __call__(self, *args, **kwargs):
                 try:
                     cache.set(cache_key, value, timeout)
                 except CacheSetError:
-                    pass
+                    logger.warning(
+                        f'Error saving value to Cache for Key: {function_cache_key}',
+                        exc_info=True,
+                    )
             return value
 
         def _invalidate(self, *args, **kwargs):
@@ -156,14 +180,14 @@ def _invalidate(self, *args, **kwargs):
             :param kwargs: The kwargs passed into the original function.
             :rtype: None
             """
-            cache_key = self.create_cache_key(*args, **kwargs)
+            cache_key, _ = self.create_cache_key(*args, **kwargs)
             cache.delete(cache_key)
 
         def create_cache_key(self, *args, **kwargs):
             # Need to include the first arg (self) in the cache key
             func_name = utils.get_function_name(self.func)
             function_cache_key = utils.get_function_cache_key(func_name, args, kwargs)
             cache_key = utils.get_hashed_cache_key(function_cache_key)
-            return cache_key
+            return cache_key, function_cache_key
 
     return wrapper