Skip to content

Commit 46c7771

Browse files
committed
Clippy
1 parent b5380c9 commit 46c7771

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

datafusion/physical-plan/src/joins/hash_join/stream.rs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -736,8 +736,7 @@ impl HashJoinStream {
736736
if need_produce_result_in_final(self.join_type) {
737737
let batch_offsets = build_side.left_data.batch_offsets();
738738
let mut bitmap = build_side.left_data.visited_indices_bitmap().lock();
739-
left_indices.iter().flatten().for_each(|x| {
740-
let packed = x as u64;
739+
left_indices.iter().flatten().for_each(|packed| {
741740
let batch_idx = (packed >> 32) as usize;
742741
let row_idx = (packed & 0xFFFFFFFF) as usize;
743742
let flat = batch_offsets[batch_idx] + row_idx;

datafusion/physical-plan/src/joins/utils.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -910,6 +910,7 @@ pub(crate) fn get_final_indices_from_bit_map(
910910
(left_indices, right_indices)
911911
}
912912

913+
#[expect(clippy::too_many_arguments)]
913914
pub(crate) fn apply_join_filter_to_indices(
914915
build_batches: &[RecordBatch],
915916
probe_batch: &RecordBatch,
@@ -1878,7 +1879,7 @@ pub(super) fn equal_rows_arr(
18781879

18791880
let mut equal: BooleanArray = eq_dyn_null(&arr_left, &arr_right, null_equality)?;
18801881

1881-
for key_idx in 1..num_keys {
1882+
for (key_idx, <item>) in right_arrays.iter().enumerate().take(num_keys).skip(1) {
18821883
let arr_left = gather_left_key(key_idx)?;
18831884
let arr_right = take(right_arrays[key_idx].as_ref(), indices_right, None)?;
18841885
let eq_result =

0 commit comments

Comments
 (0)