Skip to content

Commit

Permalink
fix: FilterMask::from_indices requires a vec (broken after filtermask…
Browse files Browse the repository at this point in the history
… changes) (#1959)
  • Loading branch information
robert3005 authored Jan 15, 2025
1 parent f30b8dd commit a0f9c7d
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions vortex-file/src/v2/file.rs
Original file line number Diff line number Diff line change
Expand Up @@ -97,9 +97,12 @@ impl<I: IoDriver> VortexFile<I> {
let filter_mask = FilterMask::from_indices(
usize::try_from(row_range.end - row_range.start)
.vortex_expect("Split ranges are within usize"),
row_indices[start_idx..end_idx].iter().map(|&idx| {
usize::try_from(idx - row_range.start).vortex_expect("index within range")
}),
row_indices[start_idx..end_idx]
.iter()
.map(|&idx| {
usize::try_from(idx - row_range.start).vortex_expect("index within range")
})
.collect(),
);
Some(RowMask::new(filter_mask, row_range.start))
});
Expand Down

0 comments on commit a0f9c7d

Please sign in to comment.