.aggregate does not raise on bad order statement #339

What happens?

This seems to compile, but the order by i at the end seems to be doing nothing?

duckdb.query("select unnest([0, 0, 1, 1]) as i, unnest([1, 2, 3, 4]) as x").aggregate("i, first(x) as x order by i")

Given that the 'correct' statement is

duckdb.query("select unnest([0, 0, 1, 1]) as i, unnest([1, 2, 3, 4]) as x").aggregate("i, first(x order by i) as x")

it seems that the former statement should raise an error to avoid mix ups, unless it has a use that I fail to see?

To Reproduce

import duckdb
duckdb.query("select unnest([0, 0, 1, 1]) as i, unnest([1, 2, 3, 4]) as x").aggregate("i, first(x) as x order by i")

OS:

Ubuntu

DuckDB Package Version:

1.4.3

Python Version:

3.10

Full Name:

Jakob Stigenberg

Affiliation:

Qubos Systematic

What is the latest build you tested with? If possible, we recommend testing with the latest nightly build.

I have tested with a stable release

Did you include all relevant data sets for reproducing the issue?

Not applicable - the reproduction does not require a data set

Did you include all code required to reproduce the issue?

• Yes, I have

Did you include all relevant configuration to reproduce the issue?

• Yes, I have