Skip to content

Commit

Permalink
fix clippy
Browse files Browse the repository at this point in the history
  • Loading branch information
Lordworms committed Mar 8, 2024
1 parent 8c97b9a commit c070b79
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions datafusion/physical-expr/src/window/lead_lag.rs
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,6 @@ fn evaluate_all_with_ignore_null(
let valid_indices: Vec<usize> = (0..array.len())
.filter(|&index| array.is_valid(index))
.collect();

let new_array_results: Result<Vec<_>, DataFusionError> = (0..array.len())
.map(|id| {
let direction = is_lag ^ (offset > 0);
Expand Down Expand Up @@ -205,7 +204,7 @@ fn evaluate_all_with_ignore_null(
.collect();

let new_array = new_array_results?;
Ok(ScalarValue::iter_to_array(new_array.into_iter())?)
ScalarValue::iter_to_array(new_array.into_iter())
}
// TODO: change the original arrow::compute::kernels::window::shift impl to support an optional default value
fn shift_with_default_value(
Expand Down

0 comments on commit c070b79

Please sign in to comment.