razzius commented on issue #13538: URL: https://github.com/apache/superset/issues/13538#issuecomment-794618476
@nytai the chart data, whether viewed from a dashboard or the explore view. I think one thing that was ambiguous in my description is this: the error that I expect to see is a Presto error, since the logged in user does not have access to the underlying data. Here's the error that we should see: <img width="809" alt="image" src="https://user-images.githubusercontent.com/2244895/110553874-553cdd00-80ff-11eb-9d98-b61a5c04f6a5.png";> I'm realizing it's a nontrivial issue, since Presto is totally left out of the picture when the result is cached. The requesting user could be incorporated into the cache key, but that wouldn't work with use cases such as Celery requesting data so that it is cached. ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
![https://user-images.githubusercontent.com/2244895/110553874-553cdd00-80ff-11eb-9d98-b61a5c04f6a5.png"](https://user-images.githubusercontent.com/2244895/110553874-553cdd00-80ff-11eb-9d98-b61a5c04f6a5.png"){kind=link}