You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2022/09/29 03:07:02 UTC
[GitHub] [superset] mayurnewase opened a new issue, #21635: Chart cache timeout is ignored when dataset cache is set
mayurnewase opened a new issue, #21635:
URL: https://github.com/apache/superset/issues/21635
A clear and concise description of what the bug is.
Chart cache timeout is ignored when dataset cache is set.
#### How to reproduce the bug
1. create any chart on 1 dataset
2. set dataset cache timeout to 120 seconds
3. set chart cache timeout to 5 seconds.
4. open chart, and after a minute click on update chart
5. backend gives cached data, and we see `cached a minute ago` tag on top right corner.
### Expected results
Chart cache timout should override dataset timeout setting, and return actual data from database after 5 seconds.
### Actual results
Chart cache timout setting is ignored, and dataset cache timeout is used for checking existing cached data..
#### Screenshots
If applicable, add screenshots to help explain your problem.
### Environment
(please complete the following information):
- browser type and version: chrome
- superset version: latest master (last commit sha: `71bf2673071d5db6688fbaefd4457aeeae3464bb`)
- python version: `python --version`: 3.9
- node.js version: `node -v`: 16
- any feature flags active: No
### Checklist
Make sure to follow these steps before submitting your issue - thank you!
- [ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
- [ ] I have reproduced the issue with at least the latest released version of superset.
- [ ] I have checked the issue tracker for the same issue and I haven't found one similar.
### Additional context
Add any other context about the problem here.
--
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-unsubscribe@superset.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] villebro commented on issue #21635: Chart cache timeout is ignored when dataset cache is set
Posted by GitBox <gi...@apache.org>.
villebro commented on issue #21635:
URL: https://github.com/apache/superset/issues/21635#issuecomment-1261748845
@mayurnewase Oh interesting. Apparently `buildQuery` isn't passing along the custom cache timouts from form data to the chart data request. I believe this needs to be fixed in `@superset-ui/core`. I can take a look, hoping it will be a simple fix.
--
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-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] rusackas closed issue #21635: Chart cache timeout is ignored
Posted by GitBox <gi...@apache.org>.
rusackas closed issue #21635: Chart cache timeout is ignored
URL: https://github.com/apache/superset/issues/21635
--
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-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
[GitHub] [superset] mayurnewase commented on issue #21635: Chart cache timeout is ignored when dataset cache is set
Posted by GitBox <gi...@apache.org>.
mayurnewase commented on issue #21635:
URL: https://github.com/apache/superset/issues/21635#issuecomment-1261766957
ouch, so `custom_cache_timeout` was supposed to be used even by the frontend?
I thougth it was for overriding all other timeouts while using that api in other automations.
I already added 1 pr: https://github.com/apache/superset/pull/21637
will close it.
@villebro
--
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-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org