Skip to content

Conversation

@jorgecarleitao
Copy link
Member

I.e. use collect instead of the builder, which is simpler and faster

@github-actions
Copy link

@codecov-io
Copy link

Codecov Report

Merging #9290 (4e5220e) into master (c413566) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #9290   +/-   ##
=======================================
  Coverage   81.61%   81.61%           
=======================================
  Files         215      215           
  Lines       52508    52508           
=======================================
  Hits        42854    42854           
  Misses       9654     9654           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c413566...0bdcf4b. Read the comment docs.

Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 nice

@alamb alamb changed the title ARROW-11343: [DataFusion] Simplified example with UDF. ARROW-11343: [Rust][DataFusion] Simplified example with UDF. Jan 22, 2021
@alamb alamb closed this in 457fa91 Jan 22, 2021
kszucs pushed a commit that referenced this pull request Jan 25, 2021
I.e. use `collect` instead of the builder, which is simpler and faster

Closes #9290 from jorgecarleitao/simpler_udf

Authored-by: Jorge C. Leitao <[email protected]>
Signed-off-by: Andrew Lamb <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants