-
Notifications
You must be signed in to change notification settings - Fork 205
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
Some aggregation functions need ORDER BY clause #4383
Comments
Yes agree. What's the class of functions that require the How would we encode this? We have a field on each function that specifies "include an |
In PRQL, columns are arrays, not sets as they are in SQL. So columns do have an order they are in, which means that by default all aggregation function should take ordering into account. So I think we could encode that in
|
(intention: I want a single row that is a json array, which contains "month: total" entries, sorted by total)
I think this PRQL should produce the following SQL:
The important bit is that
json_agg
has its column ORDERed BY total.This is because
from invoices | sort total
produces a relation, ordered by total, so this ordering should be accounted by the aggregation function.The text was updated successfully, but these errors were encountered: