diqiu50 opened a new pull request, #13142: URL: https://github.com/apache/gravitino/pull/13142
### What changes were proposed in this pull request? - Build the full `FUNCTION <name>(<params>) RETURNS <type> [NOT] DETERMINISTIC RETURN <body>` specification for SQL/TRINO functions instead of passing the stored body through. - Document that only SQL/TRINO implementations are exposed to Trino and show it in the function detail page. - Fix `TrinoUDFIT` catalog naming and run the Trino JUnit ITs in CI. ### Why are the changes needed? Trino parses `LanguageFunction.sql()` as a complete function specification, so a body like `n * 2` fails and breaks `SHOW FUNCTIONS` and invocation for the whole schema. Fix: #13139 Fix: #13140 Fix: #13141 ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Unit tests in `TestGravitinoMetadataFunction`; `TrinoUDFIT` extended with the bare-expression case and now run in CI. -- 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]
