You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by mi...@apache.org on 2019/06/01 01:28:19 UTC

[incubator-superset] 05/13: fix: cache issue for api/v1/query (#7507)

This is an automated email from the ASF dual-hosted git repository.

michellet pushed a commit to branch release--0.33
in repository https://gitbox.apache.org/repos/asf/incubator-superset.git

commit 5d1df0835cb0a0aa22e6a163fed9c60ffb7a3e18
Author: Conglei <sh...@gmail.com>
AuthorDate: Tue May 14 14:21:05 2019 -0700

    fix: cache issue for api/v1/query (#7507)
    
    
    (cherry picked from commit fda10026548169b507a2ccad3049bd3305c7978b)
---
 superset/common/query_context.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/superset/common/query_context.py b/superset/common/query_context.py
index bb91666..fd6298d 100644
--- a/superset/common/query_context.py
+++ b/superset/common/query_context.py
@@ -206,16 +206,16 @@ class QueryContext:
                         df=df if df is not None else None,
                         query=query,
                     )
-                    cache_value = pkl.dumps(
+                    cache_binary = pkl.dumps(
                         cache_value, protocol=pkl.HIGHEST_PROTOCOL)
 
                     logging.info('Caching {} chars at key {}'.format(
-                        len(cache_value), cache_key))
+                        len(cache_binary), cache_key))
 
                     stats_logger.incr('set_cache_key')
                     cache.set(
                         cache_key,
-                        cache_value,
+                        cache_value=cache_binary,
                         timeout=self.cache_timeout)
                 except Exception as e:
                     # cache.set call can fail if the backend is down or if