[GitHub] [superset] michael-s-molina commented on pull request #13802: feat(nvd3-viz): don't return empty time-series

[GitBox]

michael-s-molina commented on pull request #13802:
URL: https://github.com/apache/superset/pull/13802#issuecomment-809303445


   > should we even allow users to plot 20k series in any chart? Maybe we can 
set a hard limit on number of series allowed or show some warnings before 
plotting (only on the Explore page) if there are too many series?
   
   I agree. Without any limit, a user can freeze the application without a 
proper query. Other BI tools have default limits with the ability to increase 
them until a hard limit is reached.
   
   <img width="475" alt="Screen Shot 2021-03-29 at 8 20 36 AM" 
src="https://user-images.githubusercontent.com/70410625/112829809-07fdbc80-9068-11eb-9d38-b53aa8387e65.png";>
   <img width="398" alt="Screen Shot 2021-03-29 at 8 23 03 AM" 
src="https://user-images.githubusercontent.com/70410625/112829811-092ee980-9068-11eb-9370-122260af7180.png";>
   
   


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



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