You are viewing a plain text version of this content. The canonical link for it is here.

Posted to jira@arrow.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2021/03/13 12:38:00 UTC

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

     [ https://issues.apache.org/jira/browse/ARROW-11790?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

ASF GitHub Bot updated ARROW-11790:
-----------------------------------
    Labels: pull-request-available  (was: )

> [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
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> 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)