[jira] [Commented] (ARROW-8265) [Rust] [DataFusion] Table API collect() should not require context

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

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

Jorge commented on ARROW-8265:
------------------------------

I started some work in this direction on this branch, https://github.com/jorgecarleitao/arrow/tree/repartition, but I have not added the integration with SQL or logical plan, as I would like to know your opinions on this.

> [Rust] [DataFusion] Table API collect() should not require context
> ------------------------------------------------------------------
>
>                 Key: ARROW-8265
>                 URL: https://issues.apache.org/jira/browse/ARROW-8265
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: Rust, Rust - DataFusion
>            Reporter: Andy Grove
>            Assignee: Andy Grove
>            Priority: Major
>             Fix For: 2.0.0
>
>
> The Table API requires the context to be passed into the collect() method which leads to this odd code.
> {code:java}
> let results = ctx.table("alltypes_plain")?
>     .filter(col("c12").gt(&lit_f64(0.5)))?
>     .aggregate(vec![col("c1")], vec![min(col("c12"))])?
>     .collect(&mut ctx, 1024)?; {code}
> Since the table comes from the context, it should not be necessary to pass the context back in.



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