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

JING ZHANG commented on FLINK-23246:
------------------------------------

[~twalthr] Please have a review at 
[pr|https://github.com/apache/flink/pull/16397] when you have time, thanks a 
lot.

> Refactor the time indicator materialization
> -------------------------------------------
>
>                 Key: FLINK-23246
>                 URL: https://issues.apache.org/jira/browse/FLINK-23246
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Table SQL / Planner
>            Reporter: JING ZHANG
>            Assignee: JING ZHANG
>            Priority: Major
>              Labels: pull-request-available
>
> Refactor the time indicator materialization, including
> 1. Move `time_indicator` materialize after `logical_rewrite` phase, which is 
> closely before the physical optimization 
> 2. Port `RelTimeIndicatorConverter` from scala to Java
> 3. Refator `RelTimeIndicatorConverter` to match `FlinkLogicalRel`
> 4. split Match_ROWTIME return type converter into a separator class which 
> works before `DEFAULT_REWRITE` which contains rules to convert 
> windowProperties. 



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

Reply via email to