Skip to content

Commit 93a9a78

Browse files
committed
fix(cube): Use less stack in debug mode in logical_plan::from_proto::parse_expr
1 parent 1e1b598 commit 93a9a78

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

datafusion/proto/src/logical_plan/from_proto.rs

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -261,7 +261,10 @@ pub fn parse_expr(
261261
let scalar_value: ScalarValue = literal.try_into()?;
262262
Ok(Expr::Literal(scalar_value))
263263
}
264-
ExprType::WindowExpr(expr) => {
264+
ExprType::WindowExpr(expr) => (|expr: &Box<protobuf::WindowExprNode>,
265+
registry: &dyn FunctionRegistry,
266+
codec: &dyn LogicalExtensionCodec|
267+
-> Result<Expr, Error> {
265268
let window_function = expr
266269
.window_function
267270
.as_ref()
@@ -347,7 +350,7 @@ pub fn parse_expr(
347350
.map_err(Error::DataFusionError)
348351
}
349352
}
350-
}
353+
})(expr, registry, codec),
351354
ExprType::Alias(alias) => Ok(Expr::Alias(Alias::new(
352355
parse_required_expr(alias.expr.as_deref(), registry, "expr", codec)?,
353356
alias

0 commit comments

Comments
 (0)