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

Fabian Hueske commented on FLINK-11188:
---------------------------------------

Thanks for creating the issue [~hequn8128]. As I commented on FLINK-11172, I 
think we should not discard state of operators that are explicitly defined on a 
time range because 

1. they automatically clean up their state
2. removing state, will result in incomplete / invalid results

However, OVER ROWS windows are a bit different because they are defined on a 
row count. An OVER ROWS window on an inactive key will preserve its state 
forever and wait for more records. For these windows, state cleanup should be 
supported, IMO.

> Bounded over should not enable state retention time 
> ----------------------------------------------------
>
>                 Key: FLINK-11188
>                 URL: https://issues.apache.org/jira/browse/FLINK-11188
>             Project: Flink
>          Issue Type: Bug
>          Components: Table API & SQL
>            Reporter: Hequn Cheng
>            Assignee: Hequn Cheng
>            Priority: Major
>
> As discussed in FLINK-11172, time-based operations (GROUP BY windows, OVER 
> windows, time-windowed join, etc.) are inherently bound by time and 
> automatically clean up their state. We should not add state cleanup or TTL 
> for these operators.
> If I understand correctly, we should not add the retention logic for 
> rows-bounded operations either. I think we should disable state retention 
> logic for:
>  - ProcTimeBoundedRangeOver
>  - ProcTimeBoundedRowsOver
>  - RowTimeBoundedRangeOver
>  - RowTimeBoundedRowsOver
> Any suggestions are appreciated!



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

Reply via email to