Merge branch 'main' into feat/shuffle-testing

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())