michellethomas opened a new pull request #6244: Setting default row_limit to null for timeseries charts URL: https://github.com/apache/incubator-superset/pull/6244 This PR sets default row_limit to null for line chart, bar chart, compare chart, and area chart. Because row_limit is a new field on timeseries charts, existing charts may be affected if they have over 10,000 rows. @mistercrunch what do you think about setting the row_limit default to null? We could also set the default to 50,000 instead, it seems less likely that a chart with 50,000 rows would render very well anyways.
---------------------------------------------------------------- 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]
