Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[filter box] sort values in the filter #2367

Closed
bkyryliuk opened this issue Mar 7, 2017 · 8 comments
Closed

[filter box] sort values in the filter #2367

bkyryliuk opened this issue Mar 7, 2017 · 8 comments
Labels
airbnb Airbnb related change:backend Requires changing the backend

Comments

@bkyryliuk
Copy link
Member

allow user to sort the values in the filter

@bkyryliuk bkyryliuk added airbnb Airbnb related change:backend Requires changing the backend labels Mar 7, 2017
@bkyryliuk
Copy link
Member Author

there is a workaround via specifying the metric to be MAX or MIN

@agriffis
Copy link

agriffis commented Apr 1, 2017

It would be nice to divorce the sort from the metric, or make it selectable. Some filters (e.g. customer name) really want to be in alphabetical order. Other filters want to be in reverse alpha order, such year/quarter strings with the most recent at the top.

@mistercrunch mistercrunch changed the title Sort values in the filter [filter box] sort values in the filter Apr 2, 2017
@caseye
Copy link

caseye commented Jul 13, 2017

Agree with the workaround, but to clarify agriffis, due to the way the query is written by the system it creates a lot of wackiness. For example,


SELECT ad,
"DC_Sort"
FROM
(SELECT ad AS ad,
min(ad) AS "DC_Sort"
FROM "GGEBHART"."VW_HARDWARE_PIPELINE_UNP_ANAL"
WHERE event_date >= TO_TIMESTAMP('2017-07-06T17:44:05', 'YYYY-MM-DD"T"HH24:MI:SS.ff6')
AND event_date <= TO_TIMESTAMP('2017-07-13T17:44:05', 'YYYY-MM-DD"T"HH24:MI:SS.ff6')
GROUP BY ad
ORDER BY min(ad) DESC)
WHERE ROWNUM <= 50000

-Why does the order by default to DESC with no seeming ability to change?
-Why is it limiting Rownum to under 50,000? I mean for performance, yes... but what if I need it to look in more rows?
--Why is there an inherent time series when this has nothing to do with time?

I would actually argue that the real need is an ability to manually alter the system-generated SQL statement to make it do what you need.

@mistercrunch
Copy link
Member

Notice: this issue has been closed because it has been inactive for 283 days. Feel free to comment and request for this issue to be reopened.

@csc5k
Copy link

csc5k commented Aug 1, 2018

+1 I think it would make sense to have options within the filter slice to default sort order to alphabetic

@mosche
Copy link
Member

mosche commented Dec 24, 2018

From a dashboard consumer perspective that is in deed one of the major UX flaws of our dashboards. In some cases the current sort order is just not intuitive at all and an optional alphabetic sort order would improve things greatly!

@mosche
Copy link
Member

mosche commented Dec 24, 2018

Would be great to reopen this @mistercrunch
Also, related: #3272

@mistercrunch
Copy link
Member

Finally got around to this here #6523

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
airbnb Airbnb related change:backend Requires changing the backend
Projects
None yet
Development

No branches or pull requests

6 participants