[jira] [Commented] (ARROW-11625) [Rust] [DataFusion] Move SortExec partition check to constructor

["Hendrik Makait (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ARROW-11625?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17284454#comment-17284454 ] 

Hendrik Makait commented on ARROW-11625:
----------------------------------------

I'd love to check that out tomorrow.

> [Rust] [DataFusion] Move SortExec partition check to constructor
> ----------------------------------------------------------------
>
>                 Key: ARROW-11625
>                 URL: https://issues.apache.org/jira/browse/ARROW-11625
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: Rust - DataFusion
>            Reporter: Andy Grove
>            Priority: Major
>             Fix For: 4.0.0
>
>
> 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.
>  
> {code:java}
> if 1 != self.input.output_partitioning().partition_count() {
>     return Err(DataFusionError::Internal(
>         "SortExec requires a single input partition".to_owned(),
>     ));
> } {code}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)