Details
-
Bug
-
Status: Closed
-
Critical
-
Resolution: Fixed
-
1.9.0
Description
Currently "SHOW FUNCTIONS;" only returns catalog functions and FunctionDefinitions registered in memory, but does not include Flink built-in functions' names.
AFAIK, it's standard for "SHOW FUNCTIONS;" to show all available functions for use in queries in SQL systems like Hive, Presto, Teradata, etc, thus it includes built-in functions naturally. Besides, FunctionCatalog.lookupFunction(name) resolves calls to built-in functions, it's not feeling right to not displaying functions but can successfully resolve to them.
It seems to me that the root cause is the call stack for "SHOW FUNCTIONS;" has been a bit messy - it calls tEnv.listUserDefinedFunctions() which further calls FunctionCatalog.getUserDefinedFunctions(), and I'm not sure what's the intention of those two APIs. Are they dedicated to getting all functions, or just user defined functions excluding built-in ones?
In the end, I believe "SHOW FUNCTIONS;" should display built-in functions. To achieve that, we either need to modify and/or rename existing APIs mentioned above, or add new APIs to return all functions from FunctionCatalog.
Attachments
Issue Links
- is related to
-
FLINK-14179 Fix the description of 'SHOW FUNCTIONS' in SQL Client
- Resolved
- links to