QT/superset2
1
0
Fork 0
mirror of https://github.com/apache/superset.git synced 2026-04-25 11:04:48 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: centralize cache timeout -1 logic to prevent caching (#34654)

Browse Source
This commit is contained in:
Daniel Vaz Gaspar
2025-08-18 08:45:20 +01:00
committed by GitHub
parent 21790814db
commit 28788fd1fa
5 changed files with 19 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
superset/utils/cache.py
View File

@@ -28,6 +28,7 @@ from flask_caching.backends import NullCache
from werkzeug.wrappers import Response
from superset import db
from superset.constants import CACHE_DISABLED_TIMEOUT
from superset.extensions import cache_manager
from superset.models.cache import CacheKey
from superset.utils.hashing import md5_sha_from_dict
@@ -56,6 +57,10 @@ def set_and_log_cache(
if cache_timeout is not None
else app.config["CACHE_DEFAULT_TIMEOUT"]
)
# Skip caching if timeout is CACHE_DISABLED_TIMEOUT (no caching requested)
if timeout == CACHE_DISABLED_TIMEOUT:
return
try:
dttm = datetime.utcnow().isoformat().split(".")[0]
value = {**cache_value, "dttm": dttm}
@@ -134,7 +139,10 @@ def memoized_func(key: str, cache: Cache = cache_manager.cache) -> Callable[...,
if not force and obj is not None:
return obj
obj = f(*args, **kwargs)
cache.set(cache_key, obj, timeout=cache_timeout)
# Skip caching if timeout is CACHE_DISABLED_TIMEOUT (no caching requested)
if cache_timeout != CACHE_DISABLED_TIMEOUT:
cache.set(cache_key, obj, timeout=cache_timeout)
return obj
return wrapped_f