ggbaro opened a new issue, #25339:
URL: https://github.com/apache/superset/issues/25339

   A clear and concise description of what the bug is.
   
   #### How to reproduce the bug
   
   If we have flask-caching=~1.11.1 and cachelib>9 (flask-caching did not pin 
down cahcelib dependency until 2.0, so it's very likely to happen...), we end 
up with this https://github.com/pallets-eco/flask-caching/issues/381
   
   caused by this very line:
   
https://github.com/pallets-eco/flask-caching/blob/ac13e01baa11d65c68dcba64f99ccaafd9b5419c/src/flask_caching/backends/rediscache.py#L62
   
   The fix might be as small as this (even if I'd like to test more in depth): 
https://github.com/ggbaro/superset-ggbaro/commit/185e59ea80901600d5dc88ddc1b67e1f191c9504
   Do you suggest to make a PR?
   
   ### Expected results
   
   App starts normally
   
   ### Actual results
   
   flask-caching breaks with redis cache
   
   ### Environment
   
   (please complete the following information):
   
   - browser type and version:
   - superset version: `>=2`
   - python version: `3.9 & 3.10`
   
   ### Checklist
   
   Make sure to follow these steps before submitting your issue - thank you!
   
   - [x] I have checked the superset logs for python stacktraces and included 
it here as text if there are any.
     - see  https://github.com/pallets-eco/flask-caching/issues/381
   - [x] I have reproduced the issue with at least the latest released version 
of superset.
   - [x] I have checked the issue tracker for the same issue and I haven't 
found one similar.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org
For additional commands, e-mail: notifications-h...@superset.apache.org

Reply via email to