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

Fixed reading nested stats #1240

Merged
merged 1 commit into from
Sep 2, 2022
Merged

Fixed reading nested stats #1240

merged 1 commit into from
Sep 2, 2022

Conversation

jorgecarleitao
Copy link
Owner

Closes #1239 - thanks @ritchie46 for the report!

@jorgecarleitao jorgecarleitao added the bug Something isn't working label Sep 2, 2022
@codecov
Copy link

codecov bot commented Sep 2, 2022

Codecov Report

Merging #1240 (bde8fb9) into main (cd51b66) will increase coverage by 0.11%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1240      +/-   ##
==========================================
+ Coverage   83.31%   83.42%   +0.11%     
==========================================
  Files         359      359              
  Lines       37856    37950      +94     
==========================================
+ Hits        31539    31660     +121     
+ Misses       6317     6290      -27     
Impacted Files Coverage Δ
src/io/parquet/read/statistics/list.rs 59.61% <100.00%> (ø)
src/io/ipc/read/file.rs 97.76% <0.00%> (+0.89%) ⬆️
src/array/binary/mod.rs 90.12% <0.00%> (+1.23%) ⬆️
src/io/parquet/write/mod.rs 93.50% <0.00%> (+7.15%) ⬆️
src/io/ipc/read/array/boolean.rs 98.14% <0.00%> (+7.40%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@jorgecarleitao jorgecarleitao merged commit f34eb9b into main Sep 2, 2022
@jorgecarleitao jorgecarleitao deleted the fix_nest branch September 2, 2022 05:53
@andrei-ionescu
Copy link

Thank you @jorgecarleitao!

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.

Deeply nested struct type panics on reading parquet statistics
2 participants