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

Hequn Cheng commented on FLINK-7239:
------------------------------------

hi, [~fhueske] RetractionRules will make sure that 
{{DataStreamGroupWindowAggregate}} and {{DataStreamOverAggregate}} can not 
receive purely update messages(updates without retraction). If they did, IMO, 
we may need to fix the RetractionRules, not the other way around. 

As for the {{DataStreamWindowJoin}}, we need to set needsUpdatesAsRetraction to 
true considering that it can not handle purely update messages and check that 
the input does not have retraction messages. : )

> GroupWindow and OverWindow aggregates do not check correctly for updating 
> input
> -------------------------------------------------------------------------------
>
>                 Key: FLINK-7239
>                 URL: https://issues.apache.org/jira/browse/FLINK-7239
>             Project: Flink
>          Issue Type: Bug
>          Components: Table API & SQL
>    Affects Versions: 1.3.1, 1.4.0
>            Reporter: Fabian Hueske
>
> The streaming table operators for group-window and over-window aggregates do 
> only support append-only input, i.e., no updates.
> However, the {{DataStreamGroupWindowAggregate}} {{DataStreamOverAggregate}} 
> RelNodes do only check that the input does not have retraction messages which 
> is only one possible encoding of updates.
> We should fix the RelNodes to check if the input is append-only using the 
> {{UpdatingPlanChecker}} similar to {{DataStreamWindowJoin}}.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to