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

Added number of rows read in CSV inference #765

Merged
merged 1 commit into from
Jan 14, 2022
Merged

Conversation

jorgecarleitao
Copy link
Owner

The return argument is now (Vec<Field>, usize), where the latter is the number of read rows used in the inference.

@codecov
Copy link

codecov bot commented Jan 14, 2022

Codecov Report

Merging #765 (acba926) into main (a886e35) will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #765      +/-   ##
==========================================
+ Coverage   71.01%   71.02%   +0.01%     
==========================================
  Files         315      315              
  Lines       16950    16950              
==========================================
+ Hits        12037    12039       +2     
+ Misses       4913     4911       -2     
Impacted Files Coverage Δ
src/io/csv/read/infer_schema.rs 95.45% <100.00%> (ø)
src/io/csv/read_async/infer_schema.rs 79.16% <100.00%> (ø)
src/compute/arithmetics/time.rs 26.60% <0.00%> (+0.91%) ⬆️
src/io/parquet/read/nested_utils.rs 78.43% <0.00%> (+0.98%) ⬆️

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 a886e35...acba926. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant