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

Added GrowableFixedSizeList and improved MutableFixedSizeListArray #470

Merged
merged 1 commit into from
Sep 30, 2021

Conversation

jorgecarleitao
Copy link
Owner

@jorgecarleitao jorgecarleitao commented Sep 30, 2021

So that people can use FixedSizeList inside joins and other operations that require the growable API. The MutableFixedSizeListArray was out of spec to the rest of the mutable API and was updated accordingly.

@jorgecarleitao jorgecarleitao added the enhancement An improvement to an existing feature label Sep 30, 2021
@codecov
Copy link

codecov bot commented Sep 30, 2021

Codecov Report

Merging #470 (bc764e6) into main (3c6c075) will increase coverage by 0.00%.
The diff coverage is 84.95%.

Impacted file tree graph

@@           Coverage Diff            @@
##             main     #470    +/-   ##
========================================
  Coverage   80.01%   80.02%            
========================================
  Files         371      373     +2     
  Lines       22841    23157   +316     
========================================
+ Hits        18277    18531   +254     
- Misses       4564     4626    +62     
Impacted Files Coverage Δ
src/array/growable/mod.rs 44.44% <0.00%> (-3.56%) ⬇️
src/array/growable/fixed_size_list.rs 64.70% <64.70%> (ø)
src/array/fixed_size_list/mutable.rs 53.22% <96.15%> (+16.38%) ⬆️
tests/it/array/equal/fixed_size_list.rs 100.00% <100.00%> (ø)
tests/it/array/fixed_size_list/mutable.rs 100.00% <100.00%> (ø)
tests/it/array/growable/fixed_size_list.rs 100.00% <100.00%> (ø)
tests/it/bitmap/mutable.rs 100.00% <0.00%> (ø)
src/io/ipc/write/common.rs 76.28% <0.00%> (+0.64%) ⬆️
src/bitmap/utils/slice_iterator.rs 94.02% <0.00%> (+1.49%) ⬆️
... 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 3c6c075...bc764e6. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement An improvement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant