fix: allow aggregate functions in having outside from step

Author: KKould

src/binder/expr.rs

@@ -564,11 +564,6 @@ impl<'a, T: Transaction, A: AsRef<[(&'static str, DataValue)]>> Binder<'a, '_, T
     }
 
     fn bind_function(&mut self, func: &Function) -> Result<ScalarExpression, DatabaseError> {
-        if !matches!(self.context.step_now(), QueryBindStep::From) {
-            return Err(DatabaseError::UnsupportedStmt(
-                "`TableFunction` cannot bind in non-From step".to_string(),
-            ));
-        }
         let (func_args, is_distinct) = match &func.args {
             FunctionArguments::List(args) => (
                 args.args.as_slice(),
@@ -749,6 +744,11 @@ impl<'a, T: Transaction, A: AsRef<[(&'static str, DataValue)]>> Binder<'a, '_, T
             }));
         }
         if let Some(function) = self.context.table_functions.get(&summary) {
+            if !matches!(self.context.step_now(), QueryBindStep::From) {
+                return Err(DatabaseError::UnsupportedStmt(
+                    "`TableFunction` cannot bind in non-From step".to_string(),
+                ));
+            }
             return Ok(ScalarExpression::TableFunction(TableFunction {
                 args,
                 inner: ArcTableFunctionImpl(function.clone()),

tests/slt/having.slt

@@ -32,4 +32,9 @@ select x from test group by x having max(y) = 22
 # 23
 
 statement error
-select count(x) from test group by count(x)
+select count(x) from test group by count(x)
+
+query II
+select y as b, count(*) as total from test where x > 0 group by b having count(*) > 1 order by b
+----
+2 2