LogicalPlan::replace_params_with_values should not take &Vec<_>

[kawadakk]

Describe the bug
LogicalPlan::replace_params_with_values takes &Vec<ScalarValue>, forcing the caller to collect inputs in Vec<ScalarValue> (instead of an arbitrary continuous container) with no benefits.

https://github.com/apache/arrow-datafusion/blob/aeb593f719508973fa4564a32bc2adc767c3713e/datafusion/expr/src/logical_plan/plan.rs#L630-L636

To Reproduce
N/A

Expected behavior
A common practice, enforced by the ptr_arg Clippy lint, is to pass such parameters by slice references (&[ScalarValue]).

Additional context
N/A

[HaoYang670]

Thank you, @kawadakk, for reporting this. I will try to fix it this weekend.