Skip to content

Commit d197353

Browse files
authored
Merge pull request #116 from reddit/into-to-warn-for-decier-exceptions
feat: change log level for decider exceptions
2 parents 2272c24 + ff7d3d7 commit d197353

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

reddit_decider/__init__.py

+6-6
Original file line numberDiff line numberDiff line change
@@ -414,7 +414,7 @@ def expose(
414414
except FeatureNotFoundException:
415415
return
416416
except DeciderException as exc:
417-
logger.info(str(exc))
417+
logger.error("[decider] %s", str(exc))
418418
return
419419

420420
# drop exposure for feature rollouts
@@ -820,7 +820,7 @@ def get_all_dynamic_configs(self) -> List[Dict[str, Any]]:
820820
try:
821821
values = self._internal.all_values(ctx)
822822
except DeciderException as exc:
823-
logger.info(str(exc))
823+
logger.error("[decider] %s", str(exc))
824824
return []
825825

826826
parsed_configs = []
@@ -844,7 +844,7 @@ def _get_decision(
844844
except FeatureNotFoundException:
845845
return None
846846
except DeciderException as exc:
847-
logger.info(str(exc))
847+
logger.error("[decider] %s", str(exc))
848848
return None
849849

850850
def _get_all_decisions(
@@ -857,7 +857,7 @@ def _get_all_decisions(
857857
try:
858858
return self._internal.choose_all(ctx, bucketing_field_filter)
859859
except DeciderException as exc:
860-
logger.info(str(exc))
860+
logger.error("[decider] %s %s", str(exc))
861861
return None
862862

863863
def _get_dynamic_config_value(
@@ -877,7 +877,7 @@ def _get_dynamic_config_value(
877877
logger.info(str(exc))
878878
return default
879879
except DeciderException as exc:
880-
logger.info(str(exc))
880+
logger.error("[decider] %s", str(exc))
881881
return default
882882

883883
try:
@@ -910,7 +910,7 @@ def get_experiment(self, experiment_name: str) -> Optional[ExperimentConfig]:
910910
except FeatureNotFoundException:
911911
return None
912912
except DeciderException as exc:
913-
logger.info(str(exc))
913+
logger.error("[decider] %s", str(exc))
914914
return None
915915

916916
return ExperimentConfig(

0 commit comments

Comments
 (0)