saadtajwar commented on issue #22786:
URL: https://github.com/apache/datafusion/issues/22786#issuecomment-4781314064
@gene-bordegaray Hmm, I definitely could be mistaken here but it looks like
in the below that logical range repartitioning is now supported during physical
planning:
```
async fn map_logical_node_to_physical(...) {
....
LogicalPartitioning::Range(range) => {
let sort_exprs = create_physical_sort_exprs(
range.ordering(),
input_dfschema,
execution_props,
)?;
let ordering =
LexOrdering::new(sort_exprs).ok_or_else(|| {
internal_datafusion_err!(
"Range repartitioning requires non-empty
ordering"
)
})?;
Partitioning::Range(RangePartitioning::try_new(
ordering,
range.split_points().to_vec(),
)?)
}
...
}
```
With the test
`physical_planner::tests::logical_range_repartition_plans_output_partitioning`
passing as well! Is the above what this issue was encompassing, or is there
something else that I might be missing here? Thanks again for the help! 😁
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
