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

Added support to read Avro's structs #826

Merged
merged 1 commit into from Feb 9, 2022
Merged

Added support to read Avro's structs #826

merged 1 commit into from Feb 9, 2022

Conversation

jorgecarleitao
Copy link
Owner

Also simplified some internal code

@jorgecarleitao jorgecarleitao added the feature A new feature label Feb 8, 2022
@codecov
Copy link

codecov bot commented Feb 8, 2022

Codecov Report

Merging #826 (100677b) into main (d8093bf) will increase coverage by 0.06%.
The diff coverage is 54.28%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #826      +/-   ##
==========================================
+ Coverage   71.60%   71.67%   +0.06%     
==========================================
  Files         326      326              
  Lines       17516    17640     +124     
==========================================
+ Hits        12543    12644     +101     
- Misses       4973     4996      +23     
Impacted Files Coverage Δ
src/io/avro/read/nested.rs 38.93% <31.42%> (-5.51%) ⬇️
src/io/avro/read/schema.rs 69.62% <69.23%> (+11.39%) ⬆️
src/io/avro/read/deserialize.rs 74.30% <81.81%> (-0.50%) ⬇️
src/bitmap/utils/slice_iterator.rs 87.93% <0.00%> (+1.72%) ⬆️
src/io/avro/write/serialize.rs 89.20% <0.00%> (+3.28%) ⬆️

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 d8093bf...100677b. Read the comment docs.

@jorgecarleitao jorgecarleitao merged commit a026b66 into main Feb 9, 2022
@jorgecarleitao jorgecarleitao deleted the avro_struct branch February 9, 2022 05:30
@jorgecarleitao jorgecarleitao added enhancement An improvement to an existing feature feature A new feature and removed feature A new feature enhancement An improvement to an existing feature labels Mar 6, 2022
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.

None yet

1 participant