-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Closed
Labels
bugSomething isn't workingSomething isn't working
Description
Describe the bug
Rust 1.72 was just released and Clippy has gotten pickier: https://github.com/rust-lang/rust/blob/master/RELEASES.md#version-1720-2023-08-24
To Reproduce
update rust
Run
cargo clippy --all-targets --workspace --features avro,pyarrow -- -D warningsYou'll see errors like this:
https://github.com/alamb/arrow-datafusion/actions/runs/5965191830/job/16182337693
https://github.com/apache/arrow-datafusion/actions/runs/5964521711/job/16180238082
...
error: usage of `Iterator::fold` on a type that implements `Try`
--> datafusion/expr/src/type_coercion/other.rs:31:10
|
31 | .fold(Some(expr_type.clone()), |left, right_type| match left {
| __________^
32 | | None => None,
33 | | Some(left_type) => comparison_coercion(&left_type, right_type),
34 | | })
| |__________^ help: use `try_fold` instead: `try_fold(expr_type.clone(), |left, right_type| ...)`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold
= note: `-D clippy::manual-try-fold` implied by `-D warnings`
...
Expected behavior
Clean run
Additional context
No response
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working