Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cherry pick Fix warnings introduced by Rust/Clippy 1.57.0 to active_release #1004

Merged
merged 2 commits into from Dec 5, 2021

Conversation

alamb
Copy link
Contributor

@alamb alamb commented Dec 4, 2021

Automatic cherry-pick of 9fb2a5f

* Remove needless borrows identified by clippy

https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow

* Remove muts that are no longer needed

* Derive Default instead of using an equivalent manual impl

Identified by clippy.

https://rust-lang.github.io/rust-clippy/master/index.html#derivable_impls

* Remove redundant closures

Identified by clippy.

https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure

* Allow dead code on a field Rust now identifies as never read
@github-actions github-actions bot added arrow Changes to the arrow crate parquet Changes to the parquet crate parquet-derive labels Dec 4, 2021
@codecov-commenter
Copy link

codecov-commenter commented Dec 4, 2021

Codecov Report

Merging #1004 (72d8880) into active_release (686ac18) will decrease coverage by 0.01%.
The diff coverage is 71.05%.

Impacted file tree graph

@@                Coverage Diff                 @@
##           active_release    #1004      +/-   ##
==================================================
- Coverage           82.35%   82.34%   -0.02%     
==================================================
  Files                 168      168              
  Lines               48834    48832       -2     
==================================================
- Hits                40219    40211       -8     
- Misses               8615     8621       +6     
Impacted Files Coverage Δ
arrow/src/compute/kernels/take.rs 95.22% <ø> (-0.01%) ⬇️
arrow/src/datatypes/field.rs 53.68% <0.00%> (ø)
...ng/src/flight_server_scenarios/auth_basic_proto.rs 0.00% <ø> (ø)
parquet/src/record/reader.rs 89.83% <0.00%> (-0.63%) ⬇️
arrow/src/array/equal/utils.rs 74.00% <33.33%> (ø)
arrow/src/array/transform/boolean.rs 76.92% <100.00%> (ø)
arrow/src/compute/util.rs 98.90% <100.00%> (ø)
arrow/src/datatypes/schema.rs 66.66% <100.00%> (-0.27%) ⬇️
arrow/src/ffi.rs 84.69% <100.00%> (ø)
arrow/src/ipc/writer.rs 84.12% <100.00%> (ø)
... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 686ac18...72d8880. Read the comment docs.

@alamb alamb merged commit b4507f5 into active_release Dec 5, 2021
@alamb alamb deleted the cherry_pick_9fb2a5fa branch December 5, 2021 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arrow Changes to the arrow crate parquet Changes to the parquet crate parquet-derive
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants