Github user rmetzger commented on the issue:
https://github.com/apache/flink/pull/4041
I'll update the release announcement blog post, once the CEP doc has been
updated
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user kl0u commented on the issue:
https://github.com/apache/flink/pull/4041
Thanks a lot @dawidwys for having a detailed look at it! I will integrate
your comments and then merge.
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
Github user rmetzger commented on the issue:
https://github.com/apache/flink/pull/4041
Cool, thx
---
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
Github user dawidwys commented on the issue:
https://github.com/apache/flink/pull/4041
In fact I managed to finish the review. Generally the docs look great!
Congrats on the hard work @kl0u!
@rmetzger Most of the comments I had were to the formatting/reordering, the
only big
Github user rmetzger commented on the issue:
https://github.com/apache/flink/pull/4041
@dawidwys Do you think we can merge the updated documentation today already?
I would like to put out the 1.3 release ann with a link to the CEP docs.
---
If your project is set up for it, you ca
Github user kl0u commented on the issue:
https://github.com/apache/flink/pull/4041
Thanks a lot for the thorough review @tzulitai ! I will address your
comments and then merge.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w