fix

Signed-off-by: chagelo <changeto104@gmail.com>

src/query/ast/src/parser/query.rs

@@ -316,10 +316,6 @@ impl<'a, I: Iterator<Item = WithSpan<'a, SetOperationElement>>> PrattParser<I>
                 if query.offset.is_some() {
                     return Err("ORDER BY must appear before OFFSET");
                 }
-                let order_by = order_by
-                    .into_iter()
-                    .filter(|x| matches!(x.expr, Expr::ColumnRef { .. }))
-                    .collect();
                 query.order_by = order_by;
             }
             SetOperationElement::Limit { limit } => {

src/query/sql/src/planner/binder/sort.rs

@@ -148,6 +148,10 @@ impl Binder {
                             )
                             .map_err(|e| ErrorCode::SemanticError(e.message()))?;
 
+                        if let ScalarExpr::ConstantExpr(..) = rewrite_scalar {
+                            continue;
+                        }
+
                         let column_binding =
                             if let ScalarExpr::BoundColumnRef(col) = &rewrite_scalar {
                                 col.column.clone()

tests/sqllogictests/suites/base/03_common/03_0004_select_order_by.test

@@ -132,3 +132,60 @@ SELECT number FROM numbers_mt(10) ORDER BY sum(number)
 
 statement error 1065
 SELECT number FROM numbers_mt(10) ORDER BY count(*) + 1
+
+statement ok
+CREATE TABLE t1(a int, b int);
+
+statement ok
+INSERT INTO t1 VALUES(1, 3),(2, 1), (3, 2)
+
+query I
+SELECT * from t1 order by 1;
+----
+1 3
+2 1
+3 2
+
+query I
+SELECT * from t1 order by 2;
+----
+2 1
+3 2
+1 3
+
+statement error 1065
+SELECT * from t1 order by 3;
+
+query I
+SELECT * from t1 order by t1.a;
+----
+1 3
+2 1
+3 2
+
+query I
+SELECT * from t1 order by t1.b;
+----
+2 1
+3 2
+1 3
+
+statement error 1065
+SELECT * from t1 order by t1.1;
+
+query I 
+SELECT * from t1 order by 'a';
+----
+1 3
+2 1
+3 2
+
+query I 
+SELECT * from t1 order by '1', 1;
+----
+1 3
+2 1
+3 2
+
+statement ok
+DROP TABLE if EXISTS t1