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

Added support for compressed Avro write #699

Merged
merged 1 commit into from
Dec 23, 2021
Merged

Added support for compressed Avro write #699

merged 1 commit into from
Dec 23, 2021

Conversation

jorgecarleitao
Copy link
Owner

It also adds the crc check for snap compression described in the spec.

@jorgecarleitao jorgecarleitao added the feature A new feature label Dec 22, 2021
@jorgecarleitao jorgecarleitao force-pushed the avro_comp branch 3 times, most recently from 4fca046 to ccb3d08 Compare December 22, 2021 22:34
@codecov
Copy link

codecov bot commented Dec 22, 2021

Codecov Report

Merging #699 (605ba4c) into main (f583531) will increase coverage by 0.00%.
The diff coverage is 88.57%.

Impacted file tree graph

@@           Coverage Diff            @@
##             main     #699    +/-   ##
========================================
  Coverage   70.34%   70.35%            
========================================
  Files         308      309     +1     
  Lines       16816    16952   +136     
========================================
+ Hits        11830    11927    +97     
- Misses       4986     5025    +39     
Impacted Files Coverage Δ
src/io/avro/write/block.rs 100.00% <ø> (+33.33%) ⬆️
src/io/avro/read/decompress.rs 80.00% <63.63%> (-1.82%) ⬇️
src/io/avro/write/compress.rs 100.00% <100.00%> (ø)
src/io/avro/write/mod.rs 90.90% <100.00%> (-0.40%) ⬇️
src/io/parquet/read/nested_utils.rs 77.45% <0.00%> (-0.99%) ⬇️
src/io/avro/read/schema.rs 53.68% <0.00%> (+1.05%) ⬆️
src/io/parquet/read/mod.rs 50.15% <0.00%> (+5.39%) ⬆️
src/io/avro/write/header.rs 100.00% <0.00%> (+40.00%) ⬆️

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 f583531...605ba4c. Read the comment docs.

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.

1 participant