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

Fixed edge case in reading multiple parquet pages #904

Merged
merged 2 commits into from
Mar 12, 2022
Merged

Conversation

jorgecarleitao
Copy link
Owner

@jorgecarleitao jorgecarleitao commented Mar 12, 2022

Closes #890

@jorgecarleitao jorgecarleitao added the bug Something isn't working label Mar 12, 2022
@jorgecarleitao jorgecarleitao changed the title Fixed edge case in reading parquet pages Fixed edge case in reading multiple parquet pages Mar 12, 2022
@codecov
Copy link

codecov bot commented Mar 12, 2022

Codecov Report

Merging #904 (d62a3b9) into main (4b893b7) will decrease coverage by 0.04%.
The diff coverage is 57.37%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #904      +/-   ##
==========================================
- Coverage   70.30%   70.25%   -0.05%     
==========================================
  Files         345      345              
  Lines       18832    18822      -10     
==========================================
- Hits        13239    13224      -15     
- Misses       5593     5598       +5     
Impacted Files Coverage Δ
src/io/parquet/read/deserialize/mod.rs 69.40% <30.00%> (-2.48%) ⬇️
src/io/parquet/read/deserialize/nested_utils.rs 71.14% <62.74%> (-2.13%) ⬇️

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 4b893b7...d62a3b9. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to decode parquet file with List[PrimitiveArray<f64>]
1 participant