always use mask

alamb · alamb · commit d312a8c98f37 · 2025-12-05T17:15:05.000-05:00
diff --git a/parquet/src/arrow/arrow_reader/selection.rs b/parquet/src/arrow/arrow_reader/selection.rs
@@ -200,7 +200,8 @@ impl RowSelection {
     /// Panics if any of the [`BooleanArray`] contain nulls
     pub fn from_filters(filters: &[BooleanArray]) -> Self {
         // TODO decide how to do this based on density or something??
-        Self::Selectors(RowSelectorSelection::from_filters(filters))
+        Self::Mask(BitmaskSelection::from_filters(filters))
+        //Self::Selectors(RowSelectorSelection::from_filters(filters))
     }
 
     /// Creates a [`RowSelection`] from an iterator of consecutive ranges to keep

Original file line number	Diff line number	Diff line change
`@@ -200,7 +200,8 @@ impl RowSelection {`
`200`	`200`	/// Panics if any of the [`BooleanArray`] contain nulls
`201`	`201`	`pub fn from_filters(filters: &[BooleanArray]) -> Self {`
`202`	`202`	`// TODO decide how to do this based on density or something??`
`203`		`- Self::Selectors(RowSelectorSelection::from_filters(filters))`
	`203`	`+ Self::Mask(BitmaskSelection::from_filters(filters))`
	`204`	`+ //Self::Selectors(RowSelectorSelection::from_filters(filters))`
`204`	`205`	`}`
`205`	`206`
`206`	`207`	/// Creates a [`RowSelection`] from an iterator of consecutive ranges to keep