Skip to content

Commit

Permalink
fix: timeout should be an integer (#23924)
Browse files Browse the repository at this point in the history
  • Loading branch information
betodealmeida authored May 3, 2023
1 parent 842659d commit ad19cd9
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions superset/utils/cache.py
Original file line number Diff line number Diff line change
Expand Up @@ -95,8 +95,7 @@ def view_cache_key(*args: Any, **kwargs: Any) -> str: # pylint: disable=unused-


def memoized_func(
key: Optional[str] = None,
cache: Cache = cache_manager.cache,
key: Optional[str] = None, cache: Cache = cache_manager.cache
) -> Callable[..., Any]:
"""
Decorator with configurable key and cache backend.
Expand Down Expand Up @@ -143,7 +142,7 @@ def wrapped_f(*args: Any, **kwargs: Any) -> Any:
if not kwargs.get("force") and obj is not None:
return obj
obj = f(*args, **kwargs)
cache.set(cache_key, obj, timeout=kwargs.get("cache_timeout"))
cache.set(cache_key, obj, timeout=kwargs.get("cache_timeout", 0))
return obj

return wrapped_f
Expand Down

0 comments on commit ad19cd9

Please sign in to comment.