Minor: inline unwrap_arc, rewrite_arc, rewrite_arcs

[alamb]

Which issue does this PR close?

Part of #5637

Closes #.

Rationale for this change

When profiling on main these functions show up many times

What changes are included in this PR?

Changes
Mark the functions as #[inline(always)] - I think they are not inlined without the #inline attribute as they are in different crates

This makes the profiles much easier to read and understand:

Are these changes tested?

Functionally by exisitng tests

Performance:
(performance measurements in progress)

Code size:

I also made sure this didn't bloat the binary size:

cd datafusion-cli
cargo build --release

Before this PR:
TODO

After this PR:
TODO

Are there any user-facing changes?

Hopefuly

[alamb]

🤔 I see an error now

  Running tests/tpcds_planning.rs (target/debug/deps/tpcds_planning-cfb56ce58344e128)

running 198 tests
test tpcds_logical_q12 ... ok
test tpcds_logical_q1 ... ok
test tpcds_logical_q10 ... ok

thread 'tpcds_logical_q14' has overflowed its stack
fatal runtime error: stack overflow
error: test failed, to rerun pass `-p datafusion --test tpcds_planning`

[jayzhan211]

I think inline may also cause code bloat, especially there is recursive call for unwrap_***, worth to check it too.

[alamb]

Good call @jayzhan211 -- I will double check the size of the generated code before putting this PR up for review

[alamb]

I realstically will not have time to work on this any time soon