search

apache / arrow-julia

Official Julia implementation of Apache Arrow
https://arrow.apache.org/julia/
Other
284 stars 59 forks source link

Add Tables.partitions definition for Arrow.Table #443

Closed quinnj closed 1 year ago

quinnj commented 1 year ago

We had this functionality w/ Arrow.Stream, but it's convenient and not that expensive to define it for Arrow.Table as well.

Fixes #293.

codecov-commenter commented 1 year ago

Codecov Report

Merging #443 (8c142ad) into main (3008e7f) will increase coverage by 3.92%. The diff coverage is 89.47%.

@@            Coverage Diff             @@
##             main     #443      +/-   ##
==========================================
+ Coverage   82.98%   86.90%   +3.92%     
==========================================
  Files          25       26       +1     
  Lines        3214     3298      +84     
==========================================
+ Hits         2667     2866     +199     
+ Misses        547      432     -115
Impacted Files Coverage Δ
src/table.jl 92.75% <89.47%> (+3.41%) :arrow_up:

... and 20 files with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

quinnj commented 1 year ago

Nightly failures are https://github.com/JuliaLang/julia/pull/49777, which should be resolved soon