search

apache / datafusion

Apache DataFusion SQL Query Engine
https://datafusion.apache.org/
Apache License 2.0
6.37k stars 1.2k forks source link

Refactor signatures for lpad, rpad, left, and right #13420

Open jiashenC opened 1 week ago

jiashenC commented 1 week ago

Which issue does this PR close?

Refactor signatures for lpad, rpad, left, and right. They share very similar signatures.

Closes some tasks in #13301.

What changes are included in this PR?

Signature changes.

Are these changes tested?

Existing tests.

Are there any user-facing changes?

No

Omega359 commented 1 week ago

You can run

cargo test --test sqllogictests

locally to reproduce the test failures.