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

ASF GitHub Bot commented on FLINK-7357:
---------------------------------------

Github user twalthr commented on a diff in the pull request:

    https://github.com/apache/flink/pull/4521#discussion_r135256246
  
    --- Diff: 
flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/rules/FlinkRuleSets.scala
 ---
    @@ -147,7 +147,8 @@ object FlinkRuleSets {
     
         // Transform window to LogicalWindowAggregate
         DataSetLogicalWindowAggregateRule.INSTANCE,
    -    WindowStartEndPropertiesRule.INSTANCE
    +    WindowStartEndPropertiesRule.INSTANCE,
    +    WindowStartEndPropertiesHavingRule.INSTANCE
    --- End diff --
    
    Please also add tests for the batch logical plan.


> HOP_START() HOP_END() does not work when using HAVING clause with GROUP BY 
> HOP window
> -------------------------------------------------------------------------------------
>
>                 Key: FLINK-7357
>                 URL: https://issues.apache.org/jira/browse/FLINK-7357
>             Project: Flink
>          Issue Type: Bug
>          Components: Table API & SQL
>    Affects Versions: 1.3.1
>            Reporter: Rong Rong
>            Assignee: Rong Rong
>
> The following SQL does not compile:
> {code:title=invalid_having_hop_start_sql}
> SELECT 
>   c AS k, 
>   COUNT(a) AS v, 
>   HOP_START(rowtime, INTERVAL '1' MINUTE, INTERVAL '1' MINUTE) AS 
> windowStart, 
>   HOP_END(rowtime, INTERVAL '1' MINUTE, INTERVAL '1' MINUTE) AS windowEnd 
> FROM 
>   T1 
> GROUP BY 
>   HOP(rowtime, INTERVAL '1' MINUTE, INTERVAL '1' MINUTE), 
>   c 
> HAVING 
>   SUM(b) > 1
> {code}
> While individually keeping HAVING clause or HOP_START field compiles and runs 
> without issue.
> more details: 
> https://github.com/apache/flink/compare/master...walterddr:having_does_not_work_with_hop_start_end



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

Reply via email to