search

edgedb / rfcs

RFCs for major changes to EdgeDB
Apache License 2.0
35 stars 5 forks source link

RFC 1009: Write the initial RFC for GROUP BY #46

Closed msullivan closed 2 years ago

msullivan commented 2 years ago

I use ORDER BY because I wanted the query output in a particular order for the example, and the ORDER BY makes sure that the model actually produces the output I want. I can drop the ORDER BY, but I don't actually think that would clarify anything.

1st1 commented 2 years ago

Would be nice to see a section dedicated to analysis of the SQL GROUP BY along with some conclusions why on how our GROUP BY is different/more powerful/weaker and how our solution is more composable.

msullivan commented 2 years ago

Would be nice to see a section dedicated to analysis of the SQL GROUP BY along with some conclusions why on how our GROUP BY is different/more powerful/weaker and how our solution is more composable.

I wrote something up, with some implementation notes

msullivan commented 2 years ago

I have swapped the USING and BY clauses, again, and restored essentially my initial proposal for specifying grouping sets, but now in the BY clause instead of USING.

msullivan commented 2 years ago

Should I merge it, and then we can restart any future arguing based on that?

elprans commented 2 years ago

Should I merge it

Go ahead.