Uploaded image for project: 'Apache Arrow'
  1. Apache Arrow
  2. ARROW-11625

[Rust] [DataFusion] Move SortExec partition check to constructor

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Invalid
    • None
    • None
    • Rust - DataFusion
    • None

    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(),
          ));
      } 

      Attachments

        Activity

          People

            Unassigned Unassigned
            andygrove Andy Grove
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: