Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Added read of 2-level nested lists from parquet #548

Merged
merged 2 commits into from
Oct 24, 2021
Merged

Conversation

jorgecarleitao
Copy link
Owner

Still some edge cases to cover, but the basics are in place.

@jorgecarleitao jorgecarleitao added the feature A new feature label Oct 23, 2021
@jorgecarleitao jorgecarleitao changed the title Added red of nested lists from parquet Added read of nested lists from parquet Oct 23, 2021
@codecov
Copy link

codecov bot commented Oct 23, 2021

Codecov Report

Merging #548 (643a45a) into main (b62184d) will decrease coverage by 2.03%.
The diff coverage is 79.22%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #548      +/-   ##
==========================================
- Coverage   81.27%   79.23%   -2.04%     
==========================================
  Files         380      384       +4     
  Lines       23386    25125    +1739     
==========================================
+ Hits        19006    19908     +902     
- Misses       4380     5217     +837     
Impacted Files Coverage Δ
src/io/parquet/read/mod.rs 34.02% <35.89%> (-25.58%) ⬇️
src/io/parquet/read/binary/basic.rs 63.38% <50.00%> (-20.62%) ⬇️
src/io/parquet/read/nested_utils.rs 83.16% <86.04%> (+1.07%) ⬆️
src/io/parquet/read/binary/nested.rs 54.43% <100.00%> (-32.67%) ⬇️
src/io/parquet/read/binary/utils.rs 100.00% <100.00%> (ø)
src/io/parquet/read/boolean/nested.rs 82.35% <100.00%> (-5.41%) ⬇️
src/io/parquet/read/primitive/mod.rs 51.06% <100.00%> (-32.27%) ⬇️
tests/it/io/parquet/mod.rs 96.28% <100.00%> (+0.31%) ⬆️
tests/it/io/parquet/read.rs 99.29% <100.00%> (+0.53%) ⬆️
src/io/ipc/read/deserialize.rs 38.62% <0.00%> (-54.72%) ⬇️
... and 152 more

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 b62184d...643a45a. Read the comment docs.

@jorgecarleitao jorgecarleitao marked this pull request as ready for review October 23, 2021 17:18
@jorgecarleitao jorgecarleitao changed the title Added read of nested lists from parquet Added read of 2-level nested lists from parquet Oct 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature A new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant