Github user dianfu commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r135441351
--- Diff:
flink-libraries/flink-cep/src/main/java/org/apache/flink/cep/nfa/compiler/NFACompiler.java
---
@@ -421,6 +437,15 @@ private void addStopStateToLoop
Github user dianfu commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r135440842
--- Diff:
flink-libraries/flink-cep/src/main/java/org/apache/flink/cep/nfa/compiler/NFACompiler.java
---
@@ -526,18 +551,32 @@ private boolean isPatternOptio
Github user tedyu commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r135367240
--- Diff:
flink-libraries/flink-cep/src/main/java/org/apache/flink/cep/nfa/compiler/NFACompiler.java
---
@@ -526,18 +551,32 @@ private boolean isPatternOption
Github user tedyu commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r135366609
--- Diff:
flink-libraries/flink-cep/src/main/java/org/apache/flink/cep/nfa/compiler/NFACompiler.java
---
@@ -526,18 +551,32 @@ private boolean isPatternOption
Github user tedyu commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r135366122
--- Diff:
flink-libraries/flink-cep/src/main/java/org/apache/flink/cep/nfa/compiler/NFACompiler.java
---
@@ -421,6 +437,15 @@ private void addStopStateToLoopi
Github user asfgit closed the pull request at:
https://github.com/apache/flink/pull/4296
---
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 enab
Github user dianfu commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r131804949
--- Diff:
flink-libraries/flink-cep/src/test/java/org/apache/flink/cep/nfa/GreedyITCase.java
---
@@ -0,0 +1,1068 @@
+/*
+ * Licensed to the Apache So
Github user dianfu commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r131804902
--- Diff:
flink-libraries/flink-cep/src/test/java/org/apache/flink/cep/nfa/GreedyITCase.java
---
@@ -0,0 +1,1068 @@
+/*
+ * Licensed to the Apache So
Github user dianfu commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r131804887
--- Diff:
flink-libraries/flink-cep/src/test/java/org/apache/flink/cep/nfa/GreedyITCase.java
---
@@ -0,0 +1,1068 @@
+/*
+ * Licensed to the Apache So
Github user dianfu commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r131804932
--- Diff:
flink-libraries/flink-cep/src/test/java/org/apache/flink/cep/nfa/GreedyITCase.java
---
@@ -0,0 +1,1068 @@
+/*
+ * Licensed to the Apache So
Github user dianfu commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r131804764
--- Diff:
flink-libraries/flink-cep/src/main/java/org/apache/flink/cep/nfa/compiler/NFACompiler.java
---
@@ -637,9 +675,23 @@ private boolean isPatternOption
Github user dianfu commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r131804398
--- Diff:
flink-libraries/flink-cep/src/main/java/org/apache/flink/cep/nfa/compiler/NFACompiler.java
---
@@ -386,6 +387,19 @@ private void checkPatternNameUn
Github user dawidwys commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r131629516
--- Diff:
flink-libraries/flink-cep/src/test/java/org/apache/flink/cep/nfa/GreedyITCase.java
---
@@ -0,0 +1,1068 @@
+/*
+ * Licensed to the Apache
Github user dawidwys commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r131629581
--- Diff:
flink-libraries/flink-cep/src/test/java/org/apache/flink/cep/nfa/GreedyITCase.java
---
@@ -0,0 +1,1068 @@
+/*
+ * Licensed to the Apache
Github user dawidwys commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r131629357
--- Diff:
flink-libraries/flink-cep/src/test/java/org/apache/flink/cep/nfa/GreedyITCase.java
---
@@ -0,0 +1,1068 @@
+/*
+ * Licensed to the Apache
Github user dawidwys commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r131629322
--- Diff:
flink-libraries/flink-cep/src/test/java/org/apache/flink/cep/nfa/GreedyITCase.java
---
@@ -0,0 +1,1068 @@
+/*
+ * Licensed to the Apache
Github user dawidwys commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r131628265
--- Diff:
flink-libraries/flink-cep/src/main/java/org/apache/flink/cep/nfa/compiler/NFACompiler.java
---
@@ -386,6 +387,19 @@ private void checkPatternName
Github user dawidwys commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r131628087
--- Diff:
flink-libraries/flink-cep/src/main/java/org/apache/flink/cep/nfa/compiler/NFACompiler.java
---
@@ -637,9 +675,23 @@ private boolean isPatternOpti
Github user dianfu commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r130883414
--- Diff:
flink-libraries/flink-cep/src/main/java/org/apache/flink/cep/nfa/compiler/NFACompiler.java
---
@@ -657,25 +663,34 @@ private boolean isPatternOptio
Github user dawidwys commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r130537682
--- Diff:
flink-libraries/flink-cep/src/main/java/org/apache/flink/cep/nfa/compiler/NFACompiler.java
---
@@ -657,25 +663,34 @@ private boolean isPatternOpt
Github user dianfu commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r129832284
--- Diff:
flink-libraries/flink-cep/src/test/java/org/apache/flink/cep/nfa/GreedyITCase.java
---
@@ -0,0 +1,290 @@
+/*
+ * Licensed to the Apache Sof
Github user dianfu commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r129830324
--- Diff:
flink-libraries/flink-cep/src/main/java/org/apache/flink/cep/pattern/Quantifier.java
---
@@ -105,6 +107,14 @@ public void optional() {
Github user dianfu commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r129830367
--- Diff:
flink-libraries/flink-cep/src/main/java/org/apache/flink/cep/pattern/Pattern.java
---
@@ -492,4 +506,10 @@ private void checkIfQuantifierApplied()
Github user dawidwys commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r129263762
--- Diff:
flink-libraries/flink-cep/src/test/java/org/apache/flink/cep/nfa/GreedyITCase.java
---
@@ -0,0 +1,290 @@
+/*
+ * Licensed to the Apache S
Github user dawidwys commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r129261577
--- Diff:
flink-libraries/flink-cep/src/main/java/org/apache/flink/cep/pattern/Quantifier.java
---
@@ -105,6 +107,14 @@ public void optional() {
Github user dawidwys commented on a diff in the pull request:
https://github.com/apache/flink/pull/4296#discussion_r129262270
--- Diff:
flink-libraries/flink-cep/src/main/java/org/apache/flink/cep/pattern/Pattern.java
---
@@ -492,4 +506,10 @@ private void checkIfQuantifierApplied(
GitHub user dianfu opened a pull request:
https://github.com/apache/flink/pull/4296
[FLINK-7147] [cep] Support greedy quantifier in CEP
Thanks for contributing to Apache Flink. Before you open your pull request,
please take the following check list into consideration.
If your ch
27 matches
Mail list logo