feat: added clippy::needless_pass_by_value lint rule to datafusion/expr

[Gohlub]

Which issue does this PR close?

• Closes Enforce lint rule clippy::needless_pass_by_value to datafusion-expr #18504.

Rationale for this change

Followed suggestions to not update any public-facing APIs and put the lint rule in the appropriate spot.

What changes are included in this PR?

• Add #![deny(clippy::needless_pass_by_value)] and #![cfg_attr(test, allow(clippy::needless_pass_by_value))] to lib.rs.
• Add #[allow(clippy::needless_pass_by_value)] to public functions
• fix rewrite_in_terms_of_projection() and get_exprs_except_skipped() to use references per the lint suggestion

Are these changes tested?

Yes, though the same test failed even without changes to the public APIs:
test expr_rewriter::order_by::test::rewrite_sort_cols_by_agg_alias ... FAILED
I'll append the logs for your convenience:

failures:

---- expr_rewriter::order_by::test::rewrite_sort_cols_by_agg_alias stdout ----
running: 'c1 --> c1  -- column *named* c1 that came out of the projection, (not t.c1)'
running: 'min(c2) --> "min(c2)" -- (column *named* "min(t.c2)"!)'

thread 'expr_rewriter::order_by::test::rewrite_sort_cols_by_agg_alias' (27524241) panicked at datafusion/expr/src/expr_rewriter/order_by.rs:308:13:
assertion `left == right` failed: 

input:Sort { expr: AggregateFunction(AggregateFunction { func: AggregateUDF { inner: Min { name: "min", signature: Signature { type_signature: VariadicAny, volatility: Immutable, parameter_names: None } } }, params: AggregateFunctionParams { args: [Column(Column { relation: None, name: "c2" })], distinct: false, filter: None, order_by: [], null_treatment: None } }), asc: true, nulls_first: true }
rewritten:Sort { expr: Column(Column { relation: None, name: "min(t.c2)" }), asc: true, nulls_first: true }
expected:Sort { expr: Column(Column { relation: Some(Bare { table: "min(t" }), name: "c2)" }), asc: true, nulls_first: true }

  left: Sort { expr: Column(Column { relation: None, name: "min(t.c2)" }), asc: true, nulls_first: true }
 right: Sort { expr: Column(Column { relation: Some(Bare { table: "min(t" }), name: "c2)" }), asc: true, nulls_first: true }
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

failures:
    expr_rewriter::order_by::test::rewrite_sort_cols_by_agg_alias

Are there any user-facing changes?

No, all modification were constrained to internal APIs.

[2010YOUY01]

Thank you! This is great.

It should be good to go after CI passes. It seem there is no test failing in CI 🤔 For the formatter failure you can reproduce with ./dev/rust_lint.sh locally, and fix accordingly.

[Gohlub]

Should I try and squash the changes/merges for a cleaner history or it's fine to leave it as is?

[2010YOUY01]

Should I try and squash the changes/merges for a cleaner history or it's fine to leave it as is?

It's okay to keep it as is, since this PR is relatively small and it's easy to see the whole change -- It's helpful to cleanup the commits to several atomic units for larger PRs.