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/03/25 10:58:00 UTC
[jira] [Resolved] (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 ]
Andrew Lamb resolved ARROW-11790.
---------------------------------
Fix Version/s: 4.0.0
Resolution: Fixed
Issue resolved by pull request 9703
[https://github.com/apache/arrow/pull/9703]
> [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
> Components: Rust - DataFusion
> Reporter: Andrew Lamb
> Assignee: Andrew Lamb
> Priority: Major
> Labels: pull-request-available
> Fix For: 4.0.0
>
> Time Spent: 5h 50m
> 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)