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

    https://github.com/apache/flink/pull/4046#discussion_r120755990
  
    --- Diff: docs/dev/table/sql.md ---
    @@ -217,7 +444,7 @@ Group windows are defined in the `GROUP BY` clause of a 
SQL query. Just like que
       </tbody>
     </table>
     
    -For SQL queries on streaming tables, the `time_attr` argument of the group 
window function must be one of the `rowtime()` or `proctime()` time-indicators, 
which distinguish between event or processing time, respectively. For SQL on 
batch tables, the `time_attr` argument of the group window function must be an 
attribute of type `TIMESTAMP`. 
    +For SQL queries on streaming tables, the `time_attr` argument of the group 
window function must refer to the virtual column that specifies the processing 
time or the event time. For SQL on batch tables, the `time_attr` argument of 
the group window function must be an attribute of type `TIMESTAMP`.
    --- End diff --
    
    Add a link to the time attribute section on the Table API Streaming doc 
page.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to