From 3ae9fe821996e769bb581548d13d38cc291155ea Mon Sep 17 00:00:00 2001 From: Dustin Ray Date: Wed, 19 Jun 2024 12:55:22 -0700 Subject: [PATCH] fix: panic on unsupported groupby expressions --- crates/proof-of-sql/src/sql/ast/group_by_util.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/crates/proof-of-sql/src/sql/ast/group_by_util.rs b/crates/proof-of-sql/src/sql/ast/group_by_util.rs index 6d22b752b..7477e98de 100644 --- a/crates/proof-of-sql/src/sql/ast/group_by_util.rs +++ b/crates/proof-of-sql/src/sql/ast/group_by_util.rs @@ -113,9 +113,9 @@ pub(super) fn sum_aggregate_column_by_index_counts<'a, S: Scalar>( todo!("aggregation over decimals not yet supported") } Column::Scalar(col) => sum_aggregate_slice_by_index_counts(alloc, col, counts, indexes), - Column::VarChar(_) => unimplemented!("Cannot sum varchar columns"), + Column::VarChar(_) => panic!("Cannot sum varchar columns"), Column::TimestampTZ(_, _, _) => { - unimplemented!("Cannot sum varchar columns") + panic!("Cannot sum varchar columns") } } }