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 logical types, List,Enum, Duration and Fixed. #493

Merged
merged 3 commits into from Oct 6, 2021

Conversation

jorgecarleitao
Copy link
Owner

@jorgecarleitao jorgecarleitao commented Oct 6, 2021

Also improves testing around logical types.

@jorgecarleitao jorgecarleitao added the feature A new feature label Oct 6, 2021
@codecov
Copy link

codecov bot commented Oct 6, 2021

Codecov Report

Merging #493 (a1232ff) into main (b1285e1) will increase coverage by 0.44%.
The diff coverage is 70.31%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #493      +/-   ##
==========================================
+ Coverage   79.98%   80.42%   +0.44%     
==========================================
  Files         379      381       +2     
  Lines       23262    24113     +851     
==========================================
+ Hits        18606    19393     +787     
- Misses       4656     4720      +64     
Impacted Files Coverage Δ
src/array/fixed_size_binary/mutable.rs 65.59% <0.00%> (-1.45%) ⬇️
src/io/avro/read/nested.rs 42.85% <42.85%> (ø)
src/io/avro/read/deserialize.rs 74.77% <75.96%> (-6.36%) ⬇️
src/io/avro/read/mod.rs 78.46% <92.30%> (+1.26%) ⬆️
src/io/avro/read/schema.rs 54.76% <100.00%> (+13.38%) ⬆️
tests/it/io/avro/read/mod.rs 100.00% <100.00%> (ø)
src/scalar/primitive.rs 86.36% <0.00%> (-9.10%) ⬇️
src/scalar/list.rs 86.20% <0.00%> (-3.27%) ⬇️
src/scalar/struct_.rs 92.30% <0.00%> (-1.03%) ⬇️
... and 24 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 b1285e1...a1232ff. Read the comment docs.

@jorgecarleitao jorgecarleitao changed the title Added support to read Avro List,Enum, Duration and Fixed. Added support to read Avro logical types, List,Enum, Duration and Fixed. Oct 6, 2021
@jorgecarleitao jorgecarleitao merged commit 81edcdd into main Oct 6, 2021
@jorgecarleitao jorgecarleitao deleted the avro2 branch October 6, 2021 21:31
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

2 participants