From f83114b698b5beeb3fde3f7032c43a763a691f82 Mon Sep 17 00:00:00 2001 From: laughingman7743 Date: Sun, 8 Feb 2026 13:41:50 +0900 Subject: [PATCH] Fix retry_api_call to use getEffectiveLevel() instead of logger.level logger.level returns 0 (NOTSET) when no level is explicitly set on the logger, even when it inherits a level from a parent logger. This causes after_log callbacks to not produce any visible output. Using getEffectiveLevel() respects the logger hierarchy and returns the correct inherited level. Closes #649 Co-Authored-By: Claude Opus 4.6 --- pyathena/util.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pyathena/util.py b/pyathena/util.py index db92cab1..ce3328d6 100644 --- a/pyathena/util.py +++ b/pyathena/util.py @@ -190,7 +190,7 @@ def _is_retryable(ex: BaseException) -> bool: max=config.max_delay, exp_base=config.exponential_base, ), - after=after_log(logger, logger.level) if logger else None, # type: ignore + after=after_log(logger, logger.getEffectiveLevel()) if logger else None, # type: ignore reraise=True, ) return retry(func, *args, **kwargs)