[GitHub] [superset] villebro commented on issue #16305: [sort by][time series bart chart v2]Sort by is not working for time series bar chart v2

[GitBox <gi...@apache.org>]

villebro commented 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, 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.


-- 
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: notifications-unsubscribe@superset.apache.org

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