[ARROW-11625] [Rust] [DataFusion] Move SortExec partition check to constructor

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Invalid
• Component/s: Rust - DataFusion
• External issue URL: https://github.com/apache/arrow/issues/27489
• Assignee: Unassigned
• Reporter: Andy Grove
• Created: 14/Feb/21 15:22
• Updated: 11/Jan/23 08:20
• Resolved: 26/Apr/21 13:31

Description

SortExec has the following error check at execution time and this could be moved into the try_new constructor so the error check happens at planning time instead.

 

if 1 != self.input.output_partitioning().partition_count() {
    return Err(DataFusionError::Internal(
        "SortExec requires a single input partition".to_owned(),
    ));
}