update comments

Author: anthonyp97

cache_helper/decorators.py

@@ -36,25 +36,21 @@ def wrapper(*args, **kwargs):
                 )
                 value = sentinel
 
+            # If there is an issue with our cache client deserializing the value (due to memory or some other issue),
+            # we get a None response so log anytime this happens
             if value is None:
                 logger.warning(
                     'None cache value found for cache key: {}, function cache key: {}, value: {}'.format(
                         cache_key, function_cache_key, value
                     )
                 )
 
-            # Need to check for None as well because the cache client sends a None response at times
             if value is sentinel or value is None:
                 value = func(*args, **kwargs)
                 # Try and set the key, value pair in the cache.
                 # But if it fails on an error from the underlying
                 # cache system, handle it.
                 try:
-                    logger.info(
-                        'Setting cache key: {}, value: {} for function cache key: {}'.format(
-                            cache_key, value, function_cache_key
-                            )
-                        )
                     cache.set(cache_key, value, timeout)
                 except CacheSetError:
                     logger.warning(
@@ -104,25 +100,21 @@ def wrapper(*args, **kwargs):
                 )
                 value = sentinel
 
+            # If there is an issue with our cache client deserializing the value (due to memory or some other issue),
+            # we get a None response so log anytime this happens
             if value is None:
                 logger.warning(
                     'None cache value found for cache key: {}, function cache key: {}, value: {}'.format(
                         cache_key, function_cache_key, value
                     )
                 )
 
-            # Need to check for None as well because the cache client sends a None response at times
             if value is sentinel or value is None:
                 value = func(*args, **kwargs)
                 # Try and set the key, value pair in the cache.
                 # But if it fails on an error from the underlying
                 # cache system, handle it.
                 try:
-                    logger.info(
-                        'Setting cache key: {}, value: {} for function cache key: {}'.format(
-                            cache_key, value, function_cache_key
-                            )
-                        )
                     cache.set(cache_key, value, timeout)
                 except CacheSetError:
                     logger.warning(
@@ -193,25 +185,21 @@ def __call__(self, *args, **kwargs):
                 )
                 value = sentinel
 
+            # If there is an issue with our cache client deserializing the value (due to memory or some other issue),
+            # we get a None response so log anytime this happens
             if value is None:
                 logger.warning(
                     'None cache value found for cache key: {}, function cache key: {}, value: {}'.format(
                         cache_key, function_cache_key, value
                     )
                 )
 
-            # Need to check for None as well because the cache client sends a None response at times
             if value is sentinel or value is None:
                 value = self.func(*args, **kwargs)
                 # Try and set the key, value pair in the cache.
                 # But if it fails on an error from the underlying
                 # cache system, handle it.
                 try:
-                    logger.info(
-                        'Setting cache key: {}, value: {} for function cache key: {}'.format(
-                            cache_key, value, function_cache_key
-                            )
-                        )
                     cache.set(cache_key, value, timeout)
                 except CacheSetError:
                     logger.warning(