Skip to content

Commit

Permalink
ARROW-11881: [Rust][DataFusion] Fix clippy lint
Browse files Browse the repository at this point in the history
ARROW-11881: [Rust][DataFusion] Fix clippy lint

A linter error has appeared on master somehow:

```
error: unnecessary parentheses around `for` iterator expression
   --> datafusion/src/physical_plan/merge.rs:124:31
    |
124 |                 for part_i in (0..input_partitions) {
    |                               ^^^^^^^^^^^^^^^^^^^^^ help: remove these parentheses
    |
    = note: `-D unused-parens` implied by `-D warnings`
```

Seen on at least #9612 and #9639:

https://github.com/apache/arrow/pull/9612/checks?check_run_id=2042047472

https://github.com/apache/arrow/pull/9639/checks?check_run_id=2042649120

Closes #9642 from alamb/fix_clippy

Authored-by: Andrew Lamb <andrew@nerdnetworks.org>
Signed-off-by: Neville Dipale <nevilledips@gmail.com>
  • Loading branch information
alamb authored and nevi-me committed Mar 6, 2021
1 parent b07027e commit bfa99d9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion rust/datafusion/src/physical_plan/merge.rs
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,7 @@ impl ExecutionPlan for MergeExec {

// spawn independent tasks whose resulting streams (of batches)
// are sent to the channel for consumption.
for part_i in (0..input_partitions) {
for part_i in 0..input_partitions {
let input = self.input.clone();
let mut sender = sender.clone();
tokio::spawn(async move {
Expand Down

0 comments on commit bfa99d9

Please sign in to comment.