JoshElkind opened a new pull request, #20250: URL: https://github.com/apache/datafusion/pull/20250
## Which issue does this PR close? Closes #17083 ## Rationale for this change: Standardizes the use of Arc::clone(&ptr) over ptr.clone() across the workspace to make reference counting increments explicit and distinguishable from deep copies. This improves code readability and maintainability. ## What changes are included in this PR? - Added clippy::clone_on_ref_ptr = "deny" to workspace lints in root Cargo.toml. - Enabled lint inheritance in all sub-crates by adding [lints] workspace = true. - Removed redundant manual #![deny(clippy::clone_on_ref_ptr)] attributes from source files. - Refactored .clone() calls on Arc and Weak pointers to Arc::clone() / Weak::clone() across the workspace. - Fixed trait object coercion and missing imports (std::sync::Arc) resulting from the refactoring. ## Are these changes tested? Yes. Verified via: - cargo clippy --workspace --all-targets --all-features - cargo check --benches --examples - cargo fmt --all --check - Manual grep audit for remaining manual attributes and double-clones. ## Are there any user-facing changes? No. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
