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 2020/05/17 21:59:49 UTC

[GitHub] [incubator-superset] john-bodley edited a comment on pull request #9777: Check for empty dataframes before processing

john-bodley edited a comment on pull request #9777:
URL: https://github.com/apache/incubator-superset/pull/9777#issuecomment-629866989


   @villebro we have these empty DataFrame checks for a subset of visualizations right now. I’m not certain how this ties in with SIP-38 or whether any viz types really support no data, but it seems like we should probably short circuit the logic somewhere upstream to prevent every method requiring said check.
   
   cc @etr2460 for context who has been looking into error reporting.


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org