sadboy commented on issue #18344:
URL: https://github.com/apache/datafusion/issues/18344#issuecomment-3462494999
> I would expect the create table statement to be failing already.
Yes, disallowing Null-typed columns in `CREATE TABLE` would be more SQL
conformant.
However if DF allowing Null-typed column in tables is a concious decision,
then this should be considered a regression and we should revert back to the
v48 behavior. Disallowing `ORDER BY` Null would be problematic as this
```
DataFusion CLI v50.3.0
> with
source as (
select null as col
union all
select null
union all
select null
)
select
array_agg(distinct col) within group (order by col)
from source
group by all
;
+---------------------------------------------------------------------+
| array_agg(DISTINCT source.col) ORDER BY [source.col ASC NULLS LAST] |
+---------------------------------------------------------------------+
| [NULL] |
+---------------------------------------------------------------------+
1 row(s) fetched.
Elapsed 0.010 seconds.
```
is allowed and works the same in all major SQL engines.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
