Skip to content

Commit

Permalink
fix(cube): Union serialization breaks input order
Browse files Browse the repository at this point in the history
  • Loading branch information
paveltiunov committed Nov 27, 2024
1 parent fc9ab6b commit 613263f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion datafusion/proto/src/logical_plan/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -732,7 +732,7 @@ impl AsLogicalPlan for LogicalPlanNode {
)));
}

let first = input_plans.pop().ok_or_else(|| DataFusionError::Internal(String::from(
let first = input_plans.drain(0..1).into_iter().next().ok_or_else(|| DataFusionError::Internal(String::from(
"Protobuf deserialization error, Union was require at least two input.",
)))?;
let mut builder = LogicalPlanBuilder::from(first);
Expand Down

0 comments on commit 613263f

Please sign in to comment.