villebro edited a comment on issue #16305: URL: https://github.com/apache/superset/issues/16305#issuecomment-902462165
We may need to reconsider what the expected behavior here is. AFAIK, the primary role of the sort control is to make sure that we can pick the top n series based on a certain metric (see the subquery that queries for the top 100 series based on `count` in descending order). The ordering is done correctly in the analytical database, so the data is returned in descending order based on the `count` column. However, the data in the result pane shows the data after the pivot operation which happens in the backend (after the query), where each series is given a column of their own. We can of course resort the pivoted data, but that wouldn't change how the bars are rendered. I propose we clarify that the sorting column in timeseries charts is meant for limiting series only, and potentially introduce a subsection which encapsulates all limit related controls. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
