[ https://issues.apache.org/jira/browse/FLINK-7147?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16112698#comment-16112698 ]
ASF GitHub Bot commented on FLINK-7147: --------------------------------------- Github user dawidwys commented on the issue: https://github.com/apache/flink/pull/4296 I think we are getting close to the final version. Good job! Still though have two higher level comments: - is current times().greedy() behaviour intended? For pattern `a{2, 5} b` and sequence `a1 a2 a3 a4 b` shouldn't it return just one match: `a1 a2 a3 a4 b` Instead of three as in `testGreedyTimesRange` - this feature should be documented in the docs. It should be also very clearly stated there that it does not work for `GroupPatterns` > Support greedy quantifier in CEP > -------------------------------- > > Key: FLINK-7147 > URL: https://issues.apache.org/jira/browse/FLINK-7147 > Project: Flink > Issue Type: Sub-task > Components: CEP, Table API & SQL > Reporter: Dian Fu > Assignee: Dian Fu > > Greedy quantifier will try to match the token as many times as possible. For > example, for pattern {{a b* c}} (skip till next is used) and inputs {{a b1 b2 > c}}, if the quantifier for {{b}} is greedy, it will only output {{a b1 b2 c}}. -- This message was sent by Atlassian JIRA (v6.4.14#64029)