[CALCITE-6780] AbstractSqlTester fails to build query for expression TRIM(string)

[ILuffZhe]

This patch skips extracting trimmed character for TRIM function in CheckFramework.
Hence, after abstractSqlTester.buildQuery2(), expression trim(' aAa ') will be select trim(p0) from (values (' aAa ')) as t(p0).
And expression trim(both ' ' from ' aAa ') will be select trim(both ' ' from p0) from (values (' aAa ')) as t(p0).

[mihaibudiu]

How is this related to the CheckerFramework?
I suspect that the JIRA title is wrong.
Maybe there is a problem in the AbstractSqlTester, but that's unrelated.

[ILuffZhe]

How is this related to the CheckerFramework? I suspect that the JIRA title is wrong. Maybe there is a problem in the AbstractSqlTester, but that's unrelated.

You are right this is not about CheckerFramework. I'm considering changing the summary to AbstractSqlTester failed to build query for expression TRIM(string). What do you think?

[mihaibudiu]

that is better, but I would say "fails" instead of "failed"

[mihaibudiu]

I think we can merge this after squashing

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud