mistercrunch commented on issue #5545: [bugfix] don't cache empty dataframe URL: https://github.com/apache/incubator-superset/pull/5545#issuecomment-410349895 I fully agree and can also see it both ways. This would go back to following the previous behavior. From my understanding it was not a conscious change on this particular topic, but more the byproduct of changing some of the `No data` handling. I think here/now is the good place to discuss this and call it one way or the other. I found the this as I was debugging something else and it was very confusing, probably mostly because I had the assumption that `No data` wouldn't cache.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on 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]
