PokIsemaine commented on code in PR #15772:
URL: https://github.com/apache/datafusion/pull/15772#discussion_r2073503557
##########
datafusion/sql/src/query.rs:
##########
@@ -168,7 +169,27 @@ pub(crate) fn to_order_by_exprs_with_select(
return not_impl_err!("ORDER BY INTERPOLATE is not supported");
}
match kind {
- OrderByKind::All(_) => not_impl_err!("ORDER BY ALL is not supported"),
+ OrderByKind::All(order_by_options) => {
+ let Some(exprs) = select_exprs else {
+ return Ok(vec![]);
+ };
+ let order_by_exprs = exprs
+ .iter()
+ .filter_map(|select_expr| match select_expr {
+ Expr::Column(column) => Some(OrderByExpr {
+ expr: SQLExpr::Identifier(Ident {
+ value: column.name.clone(),
+ quote_style: None,
+ span: Span::empty(),
+ }),
+ options: order_by_options.clone(),
+ with_fill: None,
+ }),
+ _ => None,
Review Comment:
Currently, it is directly filtered out. `DuckDB` is capable of handling
other types of expressions. Perhaps I can add a `TODO` comment and use
`not_impl_err!` to notify the user, then gradually add support for other types
of `SELECT` expressions in future updates.
--
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]
