tustvold opened a new issue #412: URL: https://github.com/apache/arrow-datafusion/issues/412
**Is your feature request related to a problem or challenge? Please describe what you are trying to do.** #378 sought to loosen the requirement on `SortExec` to allow it to sort within multiple partitions, preserving the partitioning of the input. However, this broke the physical planner because it relies on the `SortExec` node to express that it only supports a single partition, so that the `AddMergeExec` optimisation pass inserts a `MergeExec` in front of it. This isn't necessarily a problem as it currently isn't possible to create an `ExecutionPlan` without immediately optimising it, but it feels a little bit unusual. **Describe the solution you'd like** The PhysicalPlanner should be aware of the partitioning within the plan, and insert the MergeExec nodes, or otherwise, necessary for the plan to be correct. **Describe alternatives you've considered** The current head of #378 adds a constructor argument to `SortExec` to opt in to looser partitioning -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
