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

Added support for datatypes serde #858

Merged
merged 1 commit into from
Feb 22, 2022

Conversation

houqp
Copy link
Collaborator

@houqp houqp commented Feb 22, 2022

feature gated with serde_datatypes, close #766.

@codecov
Copy link

codecov bot commented Feb 22, 2022

Codecov Report

Merging #858 (35d949e) into main (b6a6780) will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #858   +/-   ##
=======================================
  Coverage   71.48%   71.48%           
=======================================
  Files         335      335           
  Lines       17897    17897           
=======================================
+ Hits        12793    12794    +1     
+ Misses       5104     5103    -1     
Impacted Files Coverage Δ
src/datatypes/field.rs 100.00% <ø> (ø)
src/datatypes/mod.rs 97.22% <ø> (ø)
src/datatypes/physical_type.rs 100.00% <ø> (ø)
src/datatypes/schema.rs 33.33% <ø> (ø)
src/bitmap/utils/slice_iterator.rs 87.93% <0.00%> (+1.72%) ⬆️

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 b6a6780...35d949e. Read the comment docs.

@jorgecarleitao jorgecarleitao merged commit 22bf173 into jorgecarleitao:main Feb 22, 2022
@jorgecarleitao jorgecarleitao added the feature A new feature label Feb 22, 2022
@jorgecarleitao
Copy link
Owner

Thanks @houqp , very clean.

sydduckworth pushed a commit to mindx/arrow2 that referenced this pull request Mar 2, 2022
sydduckworth pushed a commit to mindx/arrow2 that referenced this pull request Mar 2, 2022
@jorgecarleitao jorgecarleitao changed the title add support for datatypes serde Added support for datatypes serde 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.

ser/de for datatype within scope?
2 participants