Skip to content

Commit 532b74e

Browse files
committed
Fix lints
Signed-off-by: Adam Gutglick <adamgsal@gmail.com>
1 parent f23a621 commit 532b74e

2 files changed

Lines changed: 4 additions & 5 deletions

File tree

datafusion/optimizer/src/push_down_filter.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -330,8 +330,8 @@ fn extract_or_clauses_for_join<'a>(
330330
right,
331331
}) = expr
332332
{
333-
let left_expr = extract_or_clause(left.as_ref(), &schema_cols);
334-
let right_expr = extract_or_clause(right.as_ref(), &schema_cols);
333+
let left_expr = extract_or_clause(left.as_ref(), schema_cols);
334+
let right_expr = extract_or_clause(right.as_ref(), schema_cols);
335335

336336
// If nothing can be extracted from any sub clauses, do nothing for this OR clause.
337337
if let (Some(left_expr), Some(right_expr)) = (left_expr, right_expr) {

datafusion/optimizer/src/simplify_expressions/expr_simplifier.rs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -583,8 +583,7 @@ static DUMMY_DF_SCHEMA: LazyLock<DFSchema> =
583583
static DUMMY_BATCH: LazyLock<RecordBatch> = LazyLock::new(|| {
584584
// Need a single "input" row to produce a single output row
585585
let col = new_null_array(&DataType::Null, 1);
586-
let input_batch = RecordBatch::try_new(DUMMY_SCHEMA.clone(), vec![col]).unwrap();
587-
input_batch
586+
RecordBatch::try_new(DUMMY_SCHEMA.clone(), vec![col]).unwrap()
588587
});
589588

590589
impl ConstEvaluator {
@@ -717,7 +716,7 @@ impl ConstEvaluator {
717716
false => Some(FieldMetadata::from(m)),
718717
}
719718
});
720-
let col_val = match phys_expr.evaluate(&&DUMMY_BATCH) {
719+
let col_val = match phys_expr.evaluate(&DUMMY_BATCH) {
721720
Ok(v) => v,
722721
Err(err) => return ConstSimplifyResult::SimplifyRuntimeError(err, expr),
723722
};

0 commit comments

Comments
 (0)