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

Added support for reading binary from CSV #337

Merged
merged 1 commit into from
Aug 25, 2021
Merged

Conversation

jorgecarleitao
Copy link
Owner

Useful when the CSV has fields with non-utf8 entries that are not to be discarded.

@jorgecarleitao jorgecarleitao added the feature A new feature label Aug 24, 2021
@codecov
Copy link

codecov bot commented Aug 24, 2021

Codecov Report

Merging #337 (6c3aa4b) into main (f79ae3e) will increase coverage by 0.01%.
The diff coverage is 91.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #337      +/-   ##
==========================================
+ Coverage   80.97%   80.99%   +0.01%     
==========================================
  Files         326      326              
  Lines       21167    21179      +12     
==========================================
+ Hits        17141    17154      +13     
+ Misses       4026     4025       -1     
Impacted Files Coverage Δ
src/io/csv/read/deserialize.rs 57.42% <80.00%> (+1.17%) ⬆️
tests/it/io/csv/read.rs 100.00% <100.00%> (ø)
src/array/equal/mod.rs 91.26% <0.00%> (+1.58%) ⬆️

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 f79ae3e...6c3aa4b. Read the comment docs.

@jorgecarleitao jorgecarleitao merged commit a52211d into main Aug 25, 2021
@jorgecarleitao jorgecarleitao deleted the csv_binary branch August 25, 2021 05:45
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