Changing order and group

[vmx]

I'd like to propose changing the language a bit to make it a bit more regular/consistent.

Order clause

Currently it is is:

order .baz asc default=1

I'd like to change it to:

order .baz default=1 asc

Reason: In return values it's:

return {baz: .baz default=0, hammer: .hammer default=1}

So it's (kind of like a tuple). Use this pattern for the order as well.

Grouping

Currently group() is:

group(.baz order=asc) default="a"

I'd like to change it to:

group(.baz default="a" asc)

Reason: It now has the same shape as the order clause.

What do others think about making this change?

[vmx]

The different order in the order clause is already support (both ways work), so it's just a matter of changing the documentation (which is done with #54).

For the grouping part, I wasn't aware that default values also work for other aggregates like max():

# Some data has {bar: 20} and some no value defined
find {}
return {max: max(.bar) default=1};
[
{"max":20}
]

Here the new syntax max(.bar default=120) makes even more sense. As the default a value is not the default of max(), but the default of the Keypath .bar in case that field doesn't exist.