search

jorgecarleitao / parquet2

Fastest and safest Rust implementation of parquet. `unsafe` free. Integration-tested against pyarrow
Other
356 stars 59 forks source link

Removed `EncodedPage` #191

Closed jorgecarleitao closed 2 years ago

jorgecarleitao commented 2 years ago

This was an unfortunate leftover from the previous simplification - Page is structurally the same and was intended to replace EncodedPage

codecov-commenter commented 2 years ago

Codecov Report

Merging #191 (8364bf8) into main (0301708) will increase coverage by 0.23%. The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main     #191      +/-   ##
==========================================
+ Coverage   85.75%   85.98%   +0.23%     
==========================================
  Files          84       84              
  Lines        8233     8384     +151     
==========================================
+ Hits         7060     7209     +149     
- Misses       1173     1175       +2
Impacted Files Coverage Δ
src/page/mod.rs 75.09% <ø> (+0.28%) :arrow_up:
src/write/compression.rs 59.43% <66.66%> (ø)
src/compression.rs 94.09% <0.00%> (+1.65%) :arrow_up:
src/error.rs 23.21% <0.00%> (+2.16%) :arrow_up:
src/read/compression.rs 98.55% <0.00%> (+2.58%) :arrow_up:
src/read/page/reader.rs 92.07% <0.00%> (+4.50%) :arrow_up:

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