xiedeyantu commented on PR #4359:
URL: https://github.com/apache/calcite/pull/4359#issuecomment-2848926855

   At present, errors will occur during the parsing stage. If TRIM cannot 
accept empty parameters, then this should be reasonable. I also reported 
parsing errors when verifying in MySQL and PGSQL. So does this PR still need 
for the current code?
   ```
   Caused by: org.apache.calcite.sql.parser.SqlParseException: Encountered ")" 
at line 1, column 14.
   Was expecting one of:
       "ABS" ...
       "ARRAY" ...
       "AVG" ...
       "BOTH" ...
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to