[jira] [Commented] (FLINK-11936) Remove AuxiliaryConverter pull-in from calcite and fix auxiliary converter issue.

["Rong Rong (JIRA)" <ji...@apache.org>]

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

Rong Rong commented on FLINK-11936:
-----------------------------------

After some investigation. seems like our code is far deviated from the AuxiliaryConverter and it was not kept in sync with the Calcite side. Any idea what would be the best approach here?

I am keen toward making AuxiliaryConverter implementations loaded dynamically. Currently default impl is loaded into SqlToRelConverter directly without any possibility to allow any 3rd party override. what do you think [~twalthr] ?

Another approach is to restructure the current TUMBLE/HO/SESSION window properties to align with AuxiliaryConverter logic. This would require some Calcite side fixes as well since there's still a "TODO" tag on the SESSION_END branch.

> Remove AuxiliaryConverter pull-in from calcite and fix auxiliary converter issue.
> ---------------------------------------------------------------------------------
>
>                 Key: FLINK-11936
>                 URL: https://issues.apache.org/jira/browse/FLINK-11936
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Table SQL / API
>            Reporter: Rong Rong
>            Assignee: Rong Rong
>            Priority: Major
>
> AuxiliaryConverter was pulled in FLINK-6409. Since CALCITE-1761 has been fixed, we should sync back with the calcite version.
> After a quick glance, I think it is not so simple to just delete the class so I opened a follow up Jira on this issue.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)