emilk opened a new pull request, #17565: URL: https://github.com/apache/datafusion/pull/17565
* Related: https://github.com/apache/arrow-rs/pull/8290 ## Rationale for this change I believe that error messages should be as readable as possible. Aim for `rustc` more than `gcc`. `Display` is the nice, user-facing formatter. Debug is for… well, debugging. ## What changes are included in this PR? Change a bunch of `{:?}` format string to `{}`. I'm sure I missed a lot of them, because I know of no way to enforce this without * https://github.com/rust-lang/rust-clippy/issues/8581 ## Are these changes tested? I assume CI runs `cargo test` :) ## Are there any user-facing changes? Yes! Error messages should be a bit more readable now, especially once this lands: * https://github.com/apache/arrow-rs/pull/8290 -- 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]
