Skip to content

Commit

Permalink
fix: BitWidthFreq must be u64/usize
Browse files Browse the repository at this point in the history
We assume elsewhere that this statistic is a u64 (usize is
auto-converted to u64 when we convert to Scalar). I am not
entirely sure how I triggered this, but it happens on PR
#969.
  • Loading branch information
danking committed Oct 3, 2024
1 parent 40ea77c commit ab9dd6e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion vortex-array/src/stats/statsset.rs
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ impl StatsSet {
ptype.byte_width();
stats.insert(
Stat::BitWidthFreq,
vec![0; ptype.byte_width() * 8 + 1].into(),
vec![0_u64; ptype.byte_width() * 8 + 1].into(),
);
stats.insert(
Stat::TrailingZeroFreq,
Expand Down

0 comments on commit ab9dd6e

Please sign in to comment.