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

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

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

    https://github.com/apache/flink/pull/3806#discussion_r115470902
  
    --- Diff: docs/dev/table_api.md ---
    @@ -1485,14 +1485,14 @@ Joins, set operations, and non-windowed 
aggregations are not supported yet.
     
     {% top %}
     
    -### Group Windows
    +### Windows
    --- End diff --
    
    I think we should stick to `Group Windows` here and rather adapt the 
description of the Table API windows. 
    In SQL, the term `WINDOW` refers to the `OVER` windows, so I think it is 
less ambiguous if we call them `Group Windows` and `Over Windows`.


> Update the document of group-window table API&SQL
> -------------------------------------------------
>
>                 Key: FLINK-6426
>                 URL: https://issues.apache.org/jira/browse/FLINK-6426
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Documentation, Table API & SQL
>    Affects Versions: 1.3.0
>            Reporter: sunjincheng
>            Assignee: sunjincheng
>
> 1.Correct the method parameter type error in the group-window table API 
> document, change the document from ` .window([w: Window] as 'w)` to ` 
> .window([w: WindowWithoutAlias] as 'w)`
> 2. For the consistency of tableAPI and SQL, change the description of SQL 
> document from "Group Windows" to "Windows".



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

Reply via email to