Skip to content

Commit 020c652

Browse files
committed
use try_by_name for internal unions
1 parent 86ffb0c commit 020c652

1 file changed

Lines changed: 15 additions & 16 deletions

File tree

  • datafusion_iceberg/src/materialized_view/delta_queries

datafusion_iceberg/src/materialized_view/delta_queries/transform.rs

Lines changed: 15 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -82,10 +82,9 @@ pub(crate) fn delta_transform_down(
8282
LogicalPlan::Join(join) => {
8383
let inputs = transform_join(join)?.0;
8484

85-
Ok(Transformed::yes(LogicalPlan::Union(Union {
86-
inputs,
87-
schema: join.schema.clone(),
88-
})))
85+
Ok(Transformed::yes(LogicalPlan::Union(
86+
Union::try_new_by_name(inputs)?,
87+
)))
8988
}
9089
LogicalPlan::Union(union) => {
9190
let inputs = union
@@ -190,10 +189,10 @@ pub(crate) fn delta_transform_down(
190189
null_equals_null: false,
191190
}));
192191

193-
Ok(Transformed::yes(LogicalPlan::Union(Union {
194-
inputs: vec![aggregate_projection, anti_join],
195-
schema: aggregate.schema.clone(),
196-
})))
192+
let inputs = vec![aggregate_projection, anti_join];
193+
Ok(Transformed::yes(LogicalPlan::Union(
194+
Union::try_new_by_name(inputs)?,
195+
)))
197196
}
198197
LogicalPlan::TableScan(scan) => {
199198
let mut scan = scan.clone();
@@ -275,14 +274,14 @@ pub(crate) fn delta_transform_down(
275274
join_constraint: join.join_constraint,
276275
null_equals_null: join.null_equals_null,
277276
});
278-
Ok(Transformed::yes(LogicalPlan::Union(Union {
279-
inputs: vec![
280-
Arc::new(delta_delta),
281-
Arc::new(left_delta),
282-
Arc::new(right_delta),
283-
],
284-
schema: join.schema.clone(),
285-
})))
277+
let inputs = vec![
278+
Arc::new(delta_delta),
279+
Arc::new(left_delta),
280+
Arc::new(right_delta),
281+
];
282+
Ok(Transformed::yes(LogicalPlan::Union(
283+
Union::try_new_by_name(inputs)?,
284+
)))
286285
}
287286
LogicalPlan::Union(union) => {
288287
let inputs = union

0 commit comments

Comments
 (0)