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

Removed Count (parquet statistics) #1217

Merged
merged 1 commit into from
Aug 9, 2022
Merged

Removed Count (parquet statistics) #1217

merged 1 commit into from
Aug 9, 2022

Conversation

jorgecarleitao
Copy link
Owner

This level of indirection is not required (since we already have Array for dynamic typing). It makes it easier to handle statistics.

@codecov
Copy link

codecov bot commented Aug 9, 2022

Codecov Report

Merging #1217 (12617c9) into main (838deca) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1217      +/-   ##
==========================================
- Coverage   83.17%   83.17%   -0.01%     
==========================================
  Files         358      358              
  Lines       37255    37244      -11     
==========================================
- Hits        30986    30976      -10     
+ Misses       6269     6268       -1     
Impacted Files Coverage Δ
src/io/parquet/read/statistics/mod.rs 87.83% <100.00%> (-0.32%) ⬇️
src/bitmap/utils/slice_iterator.rs 98.78% <0.00%> (+1.21%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us.

@jorgecarleitao jorgecarleitao merged commit 34f7913 into main Aug 9, 2022
@jorgecarleitao jorgecarleitao deleted the simpler_stats branch August 9, 2022 18:01
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