Skip to content

Commit

Permalink
fix: panic on unsupported groupby expressions
Browse files Browse the repository at this point in the history
  • Loading branch information
Dustin-Ray committed Jun 19, 2024
1 parent b23e815 commit 3ae9fe8
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions crates/proof-of-sql/src/sql/ast/group_by_util.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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")
}
}
}
Expand Down

0 comments on commit 3ae9fe8

Please sign in to comment.