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

Bumped to parquet2 v0.4 #352

Merged
merged 1 commit into from
Aug 30, 2021
Merged

Bumped to parquet2 v0.4 #352

merged 1 commit into from
Aug 30, 2021

Conversation

jorgecarleitao
Copy link
Owner

@jorgecarleitao jorgecarleitao commented Aug 27, 2021

  • Simplifies a lot of code
  • fixes some of the decoders
  • offers async write (which this PR does not leverage)

Closes #351
Closes #354

@jorgecarleitao jorgecarleitao added the bug Something isn't working label Aug 27, 2021
@codecov
Copy link

codecov bot commented Aug 27, 2021

Codecov Report

Merging #352 (5221f5a) into main (87aa617) will decrease coverage by 0.64%.
The diff coverage is 79.68%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #352      +/-   ##
==========================================
- Coverage   81.01%   80.36%   -0.65%     
==========================================
  Files         326      326              
  Lines       21172    21258      +86     
==========================================
- Hits        17152    17084      -68     
- Misses       4020     4174     +154     
Impacted Files Coverage Δ
src/io/parquet/read/mod.rs 51.21% <ø> (ø)
tests/it/compute/cast.rs 99.32% <ø> (-0.01%) ⬇️
src/io/parquet/read/fixed_size_binary.rs 50.00% <50.00%> (+1.80%) ⬆️
src/io/parquet/read/record_batch.rs 86.66% <50.00%> (-1.50%) ⬇️
src/io/parquet/read/binary/nested.rs 86.88% <72.72%> (+0.46%) ⬆️
src/io/parquet/read/boolean/nested.rs 90.90% <72.72%> (+1.84%) ⬆️
src/io/parquet/read/primitive/dictionary.rs 78.84% <75.00%> (-0.79%) ⬇️
src/io/parquet/read/primitive/nested.rs 92.10% <75.00%> (+2.27%) ⬆️
src/io/parquet/read/binary/basic.rs 81.67% <100.00%> (-0.42%) ⬇️
src/io/parquet/read/binary/dictionary.rs 78.84% <100.00%> (-1.16%) ⬇️
... and 9 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 87aa617...5221f5a. Read the comment docs.

@jorgecarleitao jorgecarleitao added enhancement An improvement to an existing feature and removed bug Something isn't working labels Aug 28, 2021
@jorgecarleitao jorgecarleitao merged commit d8aacdb into main Aug 30, 2021
@jorgecarleitao jorgecarleitao deleted the parquet branch August 30, 2021 19:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement An improvement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make io::parquet::write::write_* return size of file in bytes parquet_read panics with index_out_of_bounds
1 participant