Minor: reduce code duplication using rewrite_expr

[alamb]

Draft as it builds on #5092

Which issue does this PR close?

Follow on to #5092

Rationale for this change

if it is easier to rewrite expressions in DataFusion, this will lower the barrier to entry to new contributors

What changes are included in this PR?

Reduces code for the common case of rewriting an expression by letting the Rust compiler capture parameters via a closure. Some rewrites (like constant folding) still need the full generality of the ExprRewriter and I did not

Are these changes tested?

covered by existing tests

Are there any user-facing changes?

No

[ursabot]

Benchmark runs are scheduled for baseline = 4c21a72 and contender = bd64527. bd64527 is a master commit associated with this PR. Results will be available as each benchmark for each run completes.
Conbench compare runs links:
[Skipped ⚠️ Benchmarking of arrow-datafusion-commits is not supported on ec2-t3-xlarge-us-east-2] ec2-t3-xlarge-us-east-2
[Skipped ⚠️ Benchmarking of arrow-datafusion-commits is not supported on test-mac-arm] test-mac-arm
[Skipped ⚠️ Benchmarking of arrow-datafusion-commits is not supported on ursa-i9-9960x] ursa-i9-9960x
[Skipped ⚠️ Benchmarking of arrow-datafusion-commits is not supported on ursa-thinkcentre-m75q] ursa-thinkcentre-m75q
Buildkite builds:
Supported benchmarks:
ec2-t3-xlarge-us-east-2: Supported benchmark langs: Python, R. Runs only benchmarks with cloud = True
test-mac-arm: Supported benchmark langs: C++, Python, R
ursa-i9-9960x: Supported benchmark langs: Python, R, JavaScript
ursa-thinkcentre-m75q: Supported benchmark langs: C++, Java