Skip to content

Commit 2ab3724

Browse files
Matt KatzMatt Katz
authored andcommitted
remove from filter
1 parent f341260 commit 2ab3724

1 file changed

Lines changed: 1 addition & 12 deletions

File tree

vortex-array/src/arrays/filter/execute/listview.rs

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -55,18 +55,7 @@ pub fn filter_listview(array: &ListViewArray, selection_mask: &Arc<MaskValues>)
5555
// - Offsets and sizes are derived from existing valid child arrays.
5656
// - Offsets and sizes have the same length (both filtered by `selection_mask`).
5757
// - Validity matches the filtered array's nullability.
58-
let new_array = unsafe {
59-
ListViewArray::new_unchecked(elements.clone(), new_offsets, new_sizes, new_validity)
60-
};
61-
62-
let kept_row_fraction = selection_mask.true_count() as f32 / array.sizes().len() as f32;
63-
if kept_row_fraction < listview::compute::REBUILD_DENSITY_THRESHOLD {
64-
new_array
65-
.rebuild(ListViewRebuildMode::MakeZeroCopyToList)
66-
.vortex_expect("ListViewArray rebuild to zero-copy List should always succeed")
67-
} else {
68-
new_array
69-
}
58+
unsafe { ListViewArray::new_unchecked(elements.clone(), new_offsets, new_sizes, new_validity) }
7059
}
7160

7261
#[cfg(test)]

0 commit comments

Comments
 (0)