flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Timo Walther (Jira)" <j...@apache.org>
Subject [jira] [Commented] (FLINK-15595) Entirely implement resolution order as FLIP-68 concept
Date Thu, 16 Jan 2020 10:41:00 GMT

    [ https://issues.apache.org/jira/browse/FLINK-15595?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17016794#comment-17016794

Timo Walther commented on FLINK-15595:

I'm also fine with dropping the core module. Or we lookup the built in function name in the
{{FlinkSqlOperatorTable}} and disable the {{FlinkSqlOperatorTable}} fallback. The lookup solution
might cause release instabilities in case we forgot mapping certain functions by name. I would
vote for a minimal solution which would be dropping the core module for now. I mean the class
{{CoreModule}} is implemented correctly but underlying layers are not ready yet.

> Entirely implement resolution order as FLIP-68 concept
> ------------------------------------------------------
>                 Key: FLINK-15595
>                 URL: https://issues.apache.org/jira/browse/FLINK-15595
>             Project: Flink
>          Issue Type: Bug
>          Components: Table SQL / API
>            Reporter: Jingsong Lee
>            Priority: Critical
>             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. This leads to the fact that the priority
of some functions is in CoreModule, and the priority of some functions is behind all modules.
This is confusing, which is not what we want to define in FLIP-68. 
> 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

View raw message