[GitHub] flink issue #2367: [FLINK-3703][cep] Add sequence matching semantics to disc...

2017-05-11 Thread mushketyk
Github user mushketyk commented on the issue: https://github.com/apache/flink/pull/2367 @kl0u Sure. --- 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

[GitHub] flink issue #2367: [FLINK-3703][cep] Add sequence matching semantics to disc...

2017-05-11 Thread kl0u
Github user kl0u commented on the issue: https://github.com/apache/flink/pull/2367 Hi @mushketyk . I am trying to clean up a bit the open PRs for the CEP library and the related JIRAs. The issue that this PR covers was already implemented as part of multiple other

[GitHub] flink issue #2367: [FLINK-3703][cep] Add sequence matching semantics to disc...

2016-11-18 Thread LordFB
Github user LordFB commented on the issue: https://github.com/apache/flink/pull/2367 Hi @mushketyk, too bad, with this missing it is kind of a gamebreaker for Flink in my Use Case. Jeah, that'd be great, if Till would show some action on this and the related PRs.

[GitHub] flink issue #2367: [FLINK-3703][cep] Add sequence matching semantics to disc...

2016-11-15 Thread mushketyk
Github user mushketyk commented on the issue: https://github.com/apache/flink/pull/2367 Hi @LordFB I don't think there is currently a way to do it in cep-flink. I am still waiting for Till's review for this PR, but he seems to be really busy with other work. Maybe

[GitHub] flink issue #2367: [FLINK-3703][cep] Add sequence matching semantics to disc...

2016-11-15 Thread LordFB
Github user LordFB commented on the issue: https://github.com/apache/flink/pull/2367 Hi @mushketyk and @tillrohrmann, are there any updates on this Pull Request? Or is there already a way to change the matching behaviour in FlinkCEP? --- If your project is set up for it,

[GitHub] flink issue #2367: [FLINK-3703][cep] Add sequence matching semantics to disc...

2016-08-20 Thread mushketyk
Github user mushketyk commented on the issue: https://github.com/apache/flink/pull/2367 Hi @tillrohrmann I've updated the PR according to your review. I had to add reverse reverse edges to buffer entries and changed the interface of the SharedBuffer a bit to move code from it to

[GitHub] flink issue #2367: [FLINK-3703][cep] Add sequence matching semantics to disc...

2016-08-19 Thread mushketyk
Github user mushketyk commented on the issue: https://github.com/apache/flink/pull/2367 @tillrohrmann Thank you for the very detailed review. I do appreciate it. I agree with your suggestions and I'll try to improve the code and write documentation in the next couple of days.

[GitHub] flink issue #2367: [FLINK-3703][cep] Add sequence matching semantics to disc...

2016-08-19 Thread tillrohrmann
Github user tillrohrmann commented on the issue: https://github.com/apache/flink/pull/2367 Thanks for your contribution and apologies for the late review @mushketyk. I think your implementation goes into the right direction and I really like the testing. I think what we