You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@arrow.apache.org by "Andrew Lamb (Jira)" <ji...@apache.org> on 2021/02/25 22:18:00 UTC

[jira] [Created] (ARROW-11790) [Rust][DataFusion] Change plan builder signature to take Vec rather than &[Expr]

Andrew Lamb created ARROW-11790:
-----------------------------------

             Summary: [Rust][DataFusion] Change plan builder signature to take Vec<Expr> rather than &[Expr]
                 Key: ARROW-11790
                 URL: https://issues.apache.org/jira/browse/ARROW-11790
             Project: Apache Arrow
          Issue Type: Sub-task
            Reporter: Andrew Lamb


Another thing to do is to change the signagure of LogicalPlanBuilder
from taking slices of owned things &[Expr] to just taking Vec<Expr> entirely 

The rationale is that at all callsites you need to have an owned vec and Datafusion is going to copy anyways, so it would better to allow the caller to give up ownership



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