pnowojski commented on a change in pull request #7440:  [FLINK-10591][table] 
Introduced functions to return time attributes from MATCH_RECOGNIZE
URL: https://github.com/apache/flink/pull/7440#discussion_r248305449
 
 

 ##########
 File path: docs/dev/table/streaming/match_recognize.md
 ##########
 @@ -902,6 +902,37 @@ One has to keep in mind that in case of the `SKIP TO 
FIRST/LAST variable` strate
 variable (e.g. for pattern `A*`). In such cases, a runtime exception will be 
thrown as the standard requires a valid row to continue the
 matching.
 
+Time attributes
+---------------
+
+In order to apply some subsequent queries on top of the `MATCH_RECOGNIZE` it 
might be required to use [time attributes](time_attributes.html). To select 
those there
 
 Review comment:
   Can you split the longer lines (maybe even in the whole file in a separate 
commit, or at least the newly added)? Longer lines are causing problems with 
resolving conflicts and make diffs less readable.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to