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

Changed DataType::FixedSize*(i32) to DataType::FixedSize*(usize) #556

Merged
merged 1 commit into from
Oct 30, 2021

Conversation

simonvandel
Copy link
Contributor

Fixes #525

@codecov
Copy link

codecov bot commented Oct 30, 2021

Codecov Report

Merging #556 (fa92b66) into main (5fc843d) will decrease coverage by 0.00%.
The diff coverage is 73.91%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #556      +/-   ##
==========================================
- Coverage   78.92%   78.92%   -0.01%     
==========================================
  Files         382      382              
  Lines       24276    24275       -1     
==========================================
- Hits        19160    19159       -1     
  Misses       5116     5116              
Impacted Files Coverage Δ
src/datatypes/mod.rs 77.61% <ø> (ø)
src/io/avro/read/schema.rs 52.38% <0.00%> (ø)
src/io/parquet/read/statistics/fixlen.rs 54.83% <0.00%> (ø)
src/array/fixed_size_binary/mutable.rs 68.88% <50.00%> (ø)
src/io/parquet/read/fixed_size_binary.rs 61.05% <50.00%> (ø)
src/io/parquet/write/mod.rs 62.31% <50.00%> (ø)
src/array/fixed_size_binary/mod.rs 50.00% <100.00%> (ø)
src/array/fixed_size_list/mod.rs 59.70% <100.00%> (ø)
src/array/growable/fixed_binary.rs 68.75% <100.00%> (ø)
src/io/ipc/convert.rs 92.46% <100.00%> (ø)
... and 6 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 5fc843d...fa92b66. Read the comment docs.

@jorgecarleitao jorgecarleitao merged commit 5776c81 into jorgecarleitao:main Oct 30, 2021
@jorgecarleitao jorgecarleitao changed the title Change DataType::FixedSize*(i32) to DataType::FixedSize*(usize) Changed DataType::FixedSize*(i32) to DataType::FixedSize*(usize) Oct 30, 2021
@jorgecarleitao
Copy link
Owner

Thanks a lot! 🙇

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.

Change DataType::FixedSize*(i32) to DataType::FixedSize*(usize)
2 participants