[jira] [Created] (FLINK-15595) Resolution Order is chaotic not FLIP-68 defined

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Created] (FLINK-15595) Resolution Order is chaotic not FLIP-68 defined

Shang Yuanchun (Jira)
Jingsong Lee created FLINK-15595:
------------------------------------

             Summary: Resolution Order is chaotic not FLIP-68 defined
                 Key: FLINK-15595
                 URL: https://issues.apache.org/jira/browse/FLINK-15595
             Project: Flink
          Issue Type: Bug
          Components: Table SQL / API
            Reporter: Jingsong Lee
             Fix For: 1.10.0


First of all, the implementation is problematic. CoreModule returns BuiltinFunctionDefinition, which cannot be resolved in FunctionCatalogOperatorTable, so it will fall back to FlinkSqlOperatorTable.

Second, the function defined by CoreModule is seriously incomplete. You can compare it with FunctionCatalogOperatorTable, a lot less.

We should:
 * We should resolve BuiltinFunctionDefinition correctly in FunctionCatalogOperatorTable.
 * CoreModule should contains all functions in FlinkSqlOperatorTable, a simple way could provided calcite wrapper to wrap all functions.
 * PlannerContext.getBuiltinSqlOperatorTable should not contains FlinkSqlOperatorTable, we should use one FunctionCatalogOperatorTable.Otherwise, there will be a lot of confusion.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)