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

Timo Walther commented on FLINK-23544:
--------------------------------------

If we want to fix this issue properly we need to wait for CALCITE-4337. We have 
put it on our internal team roadmap but lower priority, any help here would be 
highly appreciated. Esp. by people that are more familiar with the Calcite code 
base.

I would vote for reverting this change. It is not compliant with SQL and we 
cannot effort a third syntax for session windows once CALCITE-4337 is 
implemented. Users have an alternative using the old session windows for 1.14. 
But we should aim to fix this for 1.15.

> Window TVF Supports session window
> ----------------------------------
>
>                 Key: FLINK-23544
>                 URL: https://issues.apache.org/jira/browse/FLINK-23544
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Table SQL / Planner
>            Reporter: JING ZHANG
>            Assignee: JING ZHANG
>            Priority: Minor
>              Labels: pull-request-available
>             Fix For: 1.14.0
>
>
> Window TVF would support SESSION window in the following case:
>  # SESSION Window TVF followed by Window Aggregate, in this case SESSION 
> window TVF would be pulled up into WindowAggregate, so Window assigner would 
> happen in WindowAggrgeate  
>  
> *Note, SESSION window TVF only works in limited cases currently, the 
> following user cases is not supported yet:*
>  # *SESSION WINDOW TVF followed by Window JOIN*
>  # *SESSION WINDOW TVF followed by Window RANK***
> *BESIDES, SESSION window Aggregate does not support the following performance 
> improvement yet:*
>      1. Split Distinct Aggregation
>      2. Local-global Aggregation
>      3. Mini-batch Aggregate



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

Reply via email to