Skip to content

Commit 8509263

Browse files
committed
Fmt and clippy
1 parent 2c89866 commit 8509263

2 files changed

Lines changed: 8 additions & 5 deletions

File tree

datafusion/functions-aggregate/src/sum.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -458,6 +458,7 @@ fn sum_rewrite_candidate_arg(expr: &Expr) -> Option<Expr> {
458458
}
459459

460460
/// Result of trying to split an expression into an arg and constant
461+
#[expect(clippy::large_enum_variant)]
461462
#[derive(Debug, Clone)]
462463
enum SplitResult {
463464
/// if the expression is either of

datafusion/optimizer/src/simplify_expressions/simplify_exprs.rs

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -200,14 +200,16 @@ fn rewrite_aggregate_non_aggregate_aggr_expr(
200200
.map(|expr| columnize_expr(expr, inner_aggregate.as_ref()))
201201
.collect::<Result<Vec<_>>>()?;
202202

203-
Ok(Transformed::yes(LogicalPlan::Projection(Projection::try_new(
204-
projection_exprs,
205-
inner_aggregate,
206-
)?)))
203+
Ok(Transformed::yes(LogicalPlan::Projection(
204+
Projection::try_new(projection_exprs, inner_aggregate)?,
205+
)))
207206
}
208207

209208
fn is_top_level_aggregate_expr(expr: &Expr) -> bool {
210-
matches!(expr.clone().unalias_nested().data, Expr::AggregateFunction(_))
209+
matches!(
210+
expr.clone().unalias_nested().data,
211+
Expr::AggregateFunction(_)
212+
)
211213
}
212214

213215
fn aggregate_output_exprs(group_expr: &[Expr]) -> Result<Vec<Expr>> {

0 commit comments

Comments
 (0)