Uh oh!
There was an error while loading. Please reload this page.
[ISSUE-789] Add TRIM, LPAD, and RPAD string functions - #826
Conversation
914zzzz
commented
Aug 3, 2026
mingcheng
commented
Aug 4, 2026
Please check and fix the CI status before merging. |
914zzzz
commented
Aug 5, 2026
I rechecked both failed CI runs and found the actual cause: the new TRIM runtime test called Commit Local verification:
The new upstream runs are waiting for maintainer approval (they are
Please approve these workflow runs when convenient. After they complete, the PR will still require one approving review from a reviewer with write access. |
Closes#789.
What changed
LPADandRPADwithStringandBinaryStringoverloads.TRIMoperator instead of registering a duplicate UDF.TRIM,LPAD, andRPAD.ascii2str(int)function name instead of the nonexistentascii2string(int)signature.MySQL behavior comparison
NULLNULLNULLVerification
mvn -B -e -Pjdk8 -Dtest=PadTest,TrimTest -Dmaven.javadoc.skip=true install- 5 tests passed; Checkstyle and RAT passed.mvn -B -e clean test -Pjdk11 -Dtest=PadTest,TrimTest- clean compilation of 250 main and 47 test sources; 5 tests passed; Checkstyle and RAT passed.TRIMSQL query passes Calcite planning locally. Full runtime execution is covered by the Linux CI because the local Windows BRPC shutdown selects a VPN adapter address.