Skip to content

Commit

Permalink
format
Browse files Browse the repository at this point in the history
  • Loading branch information
Lordworms committed Aug 29, 2024
1 parent 1256a15 commit b5dab19
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions datafusion/substrait/src/logical_plan/consumer.rs
Original file line number Diff line number Diff line change
Expand Up @@ -259,14 +259,12 @@ pub async fn from_substrait_plan(
)?
))
},

LogicalPlan::Aggregate(a) => {
let new_aggr_exprs = rename_expressions(a.aggr_expr, a.input.schema(), &renamed_schema)?;
Ok(LogicalPlan::Aggregate(
Aggregate::try_new(a.input, a.group_expr, new_aggr_exprs)?
))
},

// There are probably more plans where we could bake things in, can add them later as needed.
// Otherwise, add a new Project to handle the renaming.
_ => Ok(LogicalPlan::Projection(
Expand All @@ -280,7 +278,6 @@ pub async fn from_substrait_plan(
)?
)),
}

}
},
None => plan_err!("Cannot parse plan relation: None")
Expand Down

0 comments on commit b5dab19

Please sign in to comment.