andygrove opened a new pull request, #1978: URL: https://github.com/apache/datafusion-ballista/pull/1978
# Which issue does this PR close? Closes #. # Rationale for this change Stable `clippy` 1.97 (released 2026-07-07) tightened the `useless_borrows_in_formatting` lint to flag `&expr` arguments passed to formatting macros (`format!`, `println!`, `log::trace!`, `info!`, `debug!`, `error!`) where the reference is redundant. As a result, the `Clippy` CI job (`ci/scripts/rust_clippy.sh`, which runs with `-D warnings`) now fails on `main` and on every open PR, on pre-existing code untouched by those PRs. # What changes are included in this PR? Drop the redundant `&` at each flagged site (21 arguments across 8 files): - `ballista/core/src/object_store.rs`, `ballista/client/src/extension.rs` - `ballista/scheduler/src/scheduler_server/grpc.rs`, `query_stage_scheduler.rs` - `ballista/executor/src/executor_process.rs`, `executor_server.rs` - `ballista-cli/src/tui/ui/main/jobs/stage_tasks_popup.rs` - `benchmarks/src/bin/tpch.rs` `ci/scripts/rust_clippy.sh` and `cargo fmt --all -- --check` both pass cleanly after the change. # Are there any user-facing changes? No. The values were already `Display`/`Debug`-formatted through the reference, so the formatted output is byte-for-byte identical. No behavior or API change. -- 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]
