[BUG]: dont panic on count(distinct) (#3481)

Eventual-Inc · Dec 4, 2024 · 751a915 · 751a915
1 parent 988ee75
commit 751a915
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/daft-sql/src/modules/aggs.rs b/src/daft-sql/src/modules/aggs.rs
@@ -103,7 +103,7 @@ fn handle_count(inputs: &[FunctionArg], planner: &SQLPlanner) -> SQLPlannerResul
 pub fn to_expr(expr: &AggExpr, args: &[ExprRef]) -> SQLPlannerResult<ExprRef> {
     match expr {
         AggExpr::Count(_, _) => unreachable!("count should be handled by by this point"),
-        AggExpr::CountDistinct(..) => todo!("count distinct"),
+        AggExpr::CountDistinct(..) => unsupported_sql_err!("COUNT(distinct ..)"),
         AggExpr::Sum(_) => {
             ensure!(args.len() == 1, "sum takes exactly one argument");
             Ok(args[0].clone().sum())