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

Added BooleanArray::from_trusted_len_values_iter_unchecked #799

Merged
merged 1 commit into from
Feb 3, 2022

Conversation

ritchie46
Copy link
Collaborator

This functionality is already exposed on MutableBitmap. This adds a thin wrapper that makes the BooleanArray api more similar to the rest.

@codecov
Copy link

codecov bot commented Feb 3, 2022

Codecov Report

Merging #799 (59f5a58) into main (777f375) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #799      +/-   ##
==========================================
- Coverage   71.50%   71.49%   -0.02%     
==========================================
  Files         321      321              
  Lines       16739    16743       +4     
==========================================
+ Hits        11970    11971       +1     
- Misses       4769     4772       +3     
Impacted Files Coverage Δ
src/array/boolean/from.rs 57.14% <0.00%> (-4.40%) ⬇️
src/array/boolean/mutable.rs 83.61% <0.00%> (-1.45%) ⬇️
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 777f375...59f5a58. Read the comment docs.

@jorgecarleitao jorgecarleitao merged commit 374b8d7 into jorgecarleitao:main Feb 3, 2022
@jorgecarleitao jorgecarleitao changed the title add BooleanArray::from_trusted_len_values_iter_unchecked Added BooleanArray::from_trusted_len_values_iter_unchecked Feb 3, 2022
@jorgecarleitao jorgecarleitao added the enhancement An improvement to an existing feature label Feb 3, 2022
@jorgecarleitao jorgecarleitao added feature A new feature and removed enhancement An improvement to an existing feature labels 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.

2 participants