[GitHub] [superset] villebro commented on pull request #15713: [DRAFT] fix: Change url in cache-warmup celery task

[GitBox <gi...@apache.org>]

villebro commented on pull request #15713:
URL: https://github.com/apache/superset/pull/15713#issuecomment-922261110


   For charts that are using the new chart data endpoint the cache key should be calculated based on the `query-context` payload in the slice metadata (this is populated automatically when the chart is saved). In the long term we will probably introduce a Node based "sidecar" for generating the query context payload on the fly (they have to be rendered in JavaScript by the `buildQuery` callback in the plugin, hence can't be generated by the Python-based backend), but this will probably not happen in the near future.
   
   @duynguyenhoang let me know if you have time to work on this and need help.


-- 
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