Skip to content

Commit

Permalink
PR suggestion comment
Browse files Browse the repository at this point in the history
  • Loading branch information
eejbyfeldt committed Sep 25, 2024
1 parent c4e7e2f commit fee8bdf
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions datafusion/physical-plan/src/aggregates/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -261,6 +261,9 @@ impl PhysicalGroupBy {
}

/// Returns the data type of the grouping id.
/// The grouping ID value is a bitmask where each set bit
/// indicates that the corresponding grouping expression is
/// null
fn grouping_id_type(&self) -> DataType {
if self.expr.len() <= 8 {
DataType::UInt8
Expand Down

0 comments on commit fee8bdf

Please sign in to comment.