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

Anton Mushin edited comment on FLINK-5435 at 3/27/17 6:48 AM:
--------------------------------------------------------------

Hi.
We have problem in FLINK-4604 with calcite rule. In PR for FLINK-4604 I use 
work around  - the copy of rule inside Flink with changes.
For fix the problem in calcite I created jira issue: CALCITE-1621
I think that will be correct clean up the FlinkAggregateReduceFunctionsRule in 
this issue. 
What do you think about it?


was (Author: anmu):
Hi.
We have problem in FLINK-5435 with calcite rule. In PR for FLINK-5435 I use 
work around  - the copy of rule inside Flink with changes.
For fix the problem in calcite I created jira issue: CALCITE-1621
I think that will be correct clean up the FlinkAggregateReduceFunctionsRule in 
this issue. 
What do you think about it?

> Cleanup the rules introduced by FLINK-5144 when calcite releases 1.12
> ---------------------------------------------------------------------
>
>                 Key: FLINK-5435
>                 URL: https://issues.apache.org/jira/browse/FLINK-5435
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Table API & SQL
>            Reporter: Kurt Young
>            Assignee: Kurt Young
>            Priority: Minor
>
> When fixing https://issues.apache.org/jira/browse/FLINK-5144, we actually 
> copied some classes from Calcite and do a quick fix in Flink. The fixing is 
> actually merged by Calcite and will be included in version 1.12, we should 
> update the Calcite version and remove the classes we copied.
> The classes we copied:
> 1. FlinkAggregateJoinTransposeRule
> 2. RelDecorrelator -> FlinkRelDecorrelator



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to