Apply suggestions from code review #4080

Job	Run time
Check License Header	2m 21s
cargo test (win64)	1h 8m 13s
Check cargo fmt	31s
cargo check	7m 30s
build with wasm-pack	3m 0s
cargo test (macos)	19m 43s
Verify MSRV (Min Supported Rust Version)	8m 58s
cargo test (macos-aarch64)	18m 38s
Verify Vendored Code	56s
cargo test datafusion-cli (amd64)	4m 5s
cargo test (amd64)	13m 4s
cargo examples (amd64)	14m 14s
clippy	6m 17s
cargo test doc (amd64)	17m 16s
cargo doc	4m 34s
Run sqllogictest with Postgres runner	3m 44s
cargo test pyarrow (amd64)	3m 10s
verify benchmark results (amd64)	13m 5s
check Cargo.toml formatting	2m 54s
check configs.md and ***_functions.md is up-to-date	3m 5s
cargo test hash collisions (amd64)	18m 36s
	3h 53m 54s

Provide feedback