Skip to content

Commit 88fe370

Browse files
committed
Revert "feat: Add debug logger"
This reverts commit 527cd47.
1 parent 527cd47 commit 88fe370

File tree

6 files changed

+0
-32
lines changed

6 files changed

+0
-32
lines changed

src/momento/__init__.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +0,0 @@
1-
import logging
2-
3-
logging.getLogger('momentosdk').addHandler(logging.NullHandler())

src/momento/_cache_service_errors_converter.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import grpc
22
from . import errors
3-
from . import _momento_logger
43
from momento_wire_types import cacheclient_pb2 as cache_client_types
54

65
__rpc_to_error = {
@@ -31,7 +30,6 @@ def convert(exception):
3130

3231

3332
def convert_ecache_result(ecache_result, message):
34-
_momento_logger.debug(f'Converting ECacheResult: {ecache_result} to error.')
3533
if (ecache_result in __ecache_result_to_error):
3634
return __ecache_result_to_error[ecache_result](message)
3735
return errors.InternalServerError(

src/momento/_momento_logger.py

Lines changed: 0 additions & 8 deletions
This file was deleted.

src/momento/cache.py

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
from . import _cache_name_interceptor
1010
from . import errors
1111
from . import cache_operation_responses as cache_sdk_resp
12-
from . import _momento_logger
1312

1413

1514
class Cache:
@@ -33,12 +32,9 @@ def __init__(self, auth_token, cache_name, endpoint, default_ttlSeconds):
3332
# resource management for calls that need get or create functionality.
3433
def _connect(self) :
3534
try:
36-
_momento_logger.debug('Initializing connection with Cache Service')
3735
self.get(uuid.uuid1().bytes)
38-
_momento_logger.debug('Success: Connection Initialized with Cache Service')
3936
return self
4037
except Exception as e:
41-
_momento_logger.debug(f'Cache Service Connect Failed with: {e}')
4238
raise _cache_service_errors_converter.convert(e) from None
4339

4440
def __enter__(self):
@@ -49,7 +45,6 @@ def __exit__(self, exc_type, exc_value, exc_traceback):
4945

5046
def set(self, key, value, ttl_seconds=None):
5147
try:
52-
_momento_logger.debug(f'Issuing a set request with key {key}')
5348
item_ttl_seconds = self._default_ttlSeconds if ttl_seconds is None else ttl_seconds
5449
self._validate_ttl(item_ttl_seconds)
5550
set_request = cache_client_types.SetRequest()
@@ -59,24 +54,19 @@ def set(self, key, value, ttl_seconds=None):
5954
value, 'Unsupported type for value: ')
6055
set_request.ttl_milliseconds = item_ttl_seconds * 1000
6156
response = self._client.Set(set_request)
62-
_momento_logger.debug(f'Set succeeded for key: {key}')
6357
return cache_sdk_resp.CacheSetResponse(response,
6458
set_request.cache_body)
6559
except Exception as e:
66-
_momento_logger.debug(f'Set failed for {key} with response: {e}')
6760
raise _cache_service_errors_converter.convert(e)
6861

6962
def get(self, key):
7063
try:
71-
_momento_logger.debug(f'Issuing a get request with key {key}')
7264
get_request = cache_client_types.GetRequest()
7365
get_request.cache_key = self._asBytes(
7466
key, 'Unsupported type for key: ')
7567
response = self._client.Get(get_request)
76-
_momento_logger.debug(f'Received a get response for {key}')
7768
return cache_sdk_resp.CacheGetResponse(response)
7869
except Exception as e:
79-
_momento_logger.debug(f'Get failed for {key} with response: {e}')
8070
raise _cache_service_errors_converter.convert(e)
8171

8272
def _asBytes(self, data, errorMessage):

src/momento/cache_operation_responses.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
from enum import Enum
22
from momento_wire_types import cacheclient_pb2 as cache_client_types
33
from . import _cache_service_errors_converter as error_converter
4-
from . import _momento_logger
54

65

76
class CacheResult(Enum):
@@ -13,7 +12,6 @@ class CacheSetResponse:
1312
def __init__(self, grpc_set_response, value):
1413
self._value = value
1514
if (grpc_set_response.result is not cache_client_types.Ok):
16-
_momento_logger.debug(f'Set received unsupported ECacheResult {grpc_set_response.result}')
1715
raise error_converter.convert_ecache_result(
1816
grpc_set_response.result, grpc_set_response.message)
1917

@@ -30,7 +28,6 @@ def __init__(self, grpc_get_response):
3028

3129
if (grpc_get_response.result is not cache_client_types.Hit
3230
and grpc_get_response.result is not cache_client_types.Miss):
33-
_momento_logger.debug(f'Get received unsupported ECacheResult: {grpc_get_response.result}')
3431
raise error_converter.convert_ecache_result(
3532
grpc_get_response.result, grpc_get_response.message)
3633

src/momento/momento.py

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
from .cache_operation_responses import CreateCacheResponse
1212
from .cache_operation_responses import DeleteCacheResponse
1313
from .cache_operation_responses import ListCachesResponse
14-
from . import _momento_logger
1514

1615

1716
class Momento:
@@ -37,22 +36,18 @@ def __exit__(self, exc_type, exc_value, exc_traceback):
3736

3837
def create_cache(self, cache_name):
3938
try:
40-
_momento_logger.debug(f'Creating cache with name: {cache_name}')
4139
request = CreateCacheRequest()
4240
request.cache_name = cache_name
4341
return CreateCacheResponse(self._client.CreateCache(request))
4442
except Exception as e:
45-
_momento_logger.debug(f'Failed to create cache: {cache_name} with exception:{e}')
4643
raise _cache_service_errors_converter.convert(e) from None
4744

4845
def delete_cache(self, cache_name):
4946
try:
50-
_momento_logger.debug(f'Deleting cache with name: {cache_name}')
5147
request = DeleteCacheRequest()
5248
request.cache_name = cache_name
5349
return DeleteCacheResponse(self._client.DeleteCache(request))
5450
except Exception as e:
55-
_momento_logger.debug(f'Failed to delete cache: {cache_name} with exception:{e}')
5651
raise _cache_service_errors_converter.convert(e) from None
5752

5853
def get_cache(self, cache_name, ttl_seconds, create_if_absent=False):
@@ -64,7 +59,6 @@ def get_cache(self, cache_name, ttl_seconds, create_if_absent=False):
6459
if (not create_if_absent):
6560
raise e
6661

67-
_momento_logger.debug(f'create_if_absent={create_if_absent}')
6862
self.create_cache(cache_name)
6963
return cache._connect()
7064

0 commit comments

Comments
 (0)