fix: Support scalar/array args for rpad/read_side_padding

[wForget]

Which issue does this PR close?

Closes #2475.

Rationale for this change

Support full scalar/array arguments for rpad/read_side_padding

What changes are included in this PR?

Use make_scalar_function to convert arguments to arrays.

How are these changes tested?

unit test

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.40%. Comparing base (f09f8af) to head (d57ea64).
⚠️ Report is 1147 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #2482      +/-   ##
============================================
+ Coverage     56.12%   58.40%   +2.28%     
- Complexity      976     1436     +460     
============================================
  Files           119      146      +27     
  Lines         11743    13510    +1767     
  Branches       2251     2350      +99     
============================================
+ Hits           6591     7891    +1300     
- Misses         4012     4389     +377     
- Partials       1140     1230      +90     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[wForget]

Failed spark test case:

[info] - SPARK-48498: always do char padding in predicates *** FAILED *** (330 milliseconds)
[info]   org.apache.spark.SparkException: Job aborted due to stage failure: Task 0 in stage 605.0 failed 1 times, most recent failure: Lost task 0.0 in stage 605.0 (TID 582) (735d43d611a7 executor driver): org.apache.comet.CometNativeException: Unsupported data type (Dictionary(Int32, Utf8), Utf8) for function rpad/read_side_padding.
[info] This issue was likely caused by a bug in DataFusion's code. Please help us to resolve this by filing a bug report in our issue tracker: https://github.com/apache/datafusion/issues
[info] 	at org.apache.comet.Native.executePlan(Native Method)

[comphead]

@coderfender as were working on rpad/lpad

[wForget]

@coderfender as were working on rpad/lpad

Thank you for the info, feel free to take over this.

[coderfender]

@wForget , If my understanding is right, you are adding support to non scalar args to rpad function is it ? I can take a stab at the PR this weekend , fix the merge conflicts and make changes necessary to match spark's behavior and I hope that is Ok ?

[github-actions]

Thank you for your contribution. Unfortunately, this pull request is stale because it has been open 60 days with no activity. Please remove the stale label or comment or this will be closed in 7 days.