betodealmeida commented on issue #7148: Celery task for warming up cache
URL: 
https://github.com/apache/incubator-superset/pull/7148#issuecomment-477690854
 
 
   Thanks for the comments, @john-bodley!
   
   > Regarding the top-n charts you may want to time bound this, i.e., the top 
viewed charts in the past month, quarter, etc.
   
   Ah, great point! Will do that.
   
   > One thing the current cache warmup doesn’t deal with is warming up 
dashboards, instead it warms up saves slices associated with dashboards which 
doesn’t take into account any dashboard filters. I wonder if there’s a way to 
add this functionality.
   
   Also a great point, for Lyft we actually need this. I'll add it.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to