Skip to content

Commit d2be0cc

Browse files
cargo fmt
1 parent 04799ee commit d2be0cc

File tree

4 files changed

+11
-17
lines changed

4 files changed

+11
-17
lines changed

datafusion/physical-expr/src/expressions/in_list.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,8 @@ use crate::physical_expr::physical_exprs_bag_equal;
3333

3434
use arrow::array::*;
3535
use arrow::buffer::{BooleanBuffer, NullBuffer};
36-
use arrow::compute::kernels::boolean::{not, or_kleene};
3736
use arrow::compute::SortOptions;
37+
use arrow::compute::kernels::boolean::{not, or_kleene};
3838
use arrow::datatypes::*;
3939
use datafusion_common::{
4040
DFSchema, Result, ScalarValue, assert_or_internal_err, exec_err,

datafusion/physical-expr/src/expressions/in_list/array_filter.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,8 @@ impl StaticFilter for ArrayStaticFilter {
5656
/// Checks if values in `v` are contained in the `in_array` using this hash set for lookup.
5757
fn contains(&self, v: &dyn Array, negated: bool) -> Result<BooleanArray> {
5858
// Null type comparisons always return null (SQL three-valued logic)
59-
if v.data_type() == &DataType::Null || self.in_array.data_type() == &DataType::Null
59+
if v.data_type() == &DataType::Null
60+
|| self.in_array.data_type() == &DataType::Null
6061
{
6162
let nulls = NullBuffer::new_null(v.len());
6263
return Ok(BooleanArray::new(

datafusion/physical-expr/src/expressions/in_list/primitive.rs

Lines changed: 6 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -142,13 +142,9 @@ impl<C: BitmapFilterConfig> BitmapFilter<C> {
142142
nulls: Option<&arrow::buffer::NullBuffer>,
143143
negated: bool,
144144
) -> BooleanArray {
145-
build_in_list_result(
146-
values.len(),
147-
nulls,
148-
self.null_count > 0,
149-
negated,
150-
|i| self.check(unsafe { *values.get_unchecked(i) }),
151-
)
145+
build_in_list_result(values.len(), nulls, self.null_count > 0, negated, |i| {
146+
self.check(unsafe { *values.get_unchecked(i) })
147+
})
152148
}
153149
}
154150

@@ -227,13 +223,9 @@ where
227223
nulls: Option<&arrow::buffer::NullBuffer>,
228224
negated: bool,
229225
) -> BooleanArray {
230-
build_in_list_result(
231-
values.len(),
232-
nulls,
233-
self.null_count > 0,
234-
negated,
235-
|i| self.check(unsafe { *values.get_unchecked(i) }),
236-
)
226+
build_in_list_result(values.len(), nulls, self.null_count > 0, negated, |i| {
227+
self.check(unsafe { *values.get_unchecked(i) })
228+
})
237229
}
238230
}
239231

datafusion/physical-expr/src/expressions/in_list/strategy.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,8 @@ use super::array_filter::StaticFilter;
3939
use super::primitive::{PrimitiveFilter, U8Config, U16Config};
4040
use super::transform::{
4141
make_bitmap_filter, make_branchless_filter, make_primitive_filter,
42-
make_utf8view_branchless_filter, make_utf8view_hash_filter, utf8view_all_short_strings,
42+
make_utf8view_branchless_filter, make_utf8view_hash_filter,
43+
utf8view_all_short_strings,
4344
};
4445

4546
// =============================================================================

0 commit comments

Comments
 (0)