[GitHub] flink issue #6076: [hotfix][docs] Specify operators behaviour on processing ...

2018-07-03 Thread eliaslevy
Github user eliaslevy commented on the issue: https://github.com/apache/flink/pull/6076 @pnowojski what about refactoring the doc into something like [this](https://docs.google.com/document/d/1b5d-hTdJQsPH3YD0zTB4ZqodinZVHFomKvt41FfUPMc/edit?usp=sharing). ---

[GitHub] flink issue #6076: [hotfix][docs] Specify operators behaviour on processing ...

2018-06-15 Thread eliaslevy
Github user eliaslevy commented on the issue: https://github.com/apache/flink/pull/6076 Something else missing from this page is a discussion of what happens to the timestamp attached to records when they go through different operations. E.g. what happens to a record timestamp when

[GitHub] flink pull request #6076: [hotfix][docs] Specify operators behaviour on proc...

2018-06-06 Thread eliaslevy
Github user eliaslevy commented on a diff in the pull request: https://github.com/apache/flink/pull/6076#discussion_r193619225 --- Diff: docs/dev/event_time.md --- @@ -213,10 +213,33 @@ arrive after the system's event time clock (as signaled by the watermarks) has a timestamp

[GitHub] flink issue #6076: [hotfix][docs] Specify operators behaviour on processing ...

2018-05-28 Thread eliaslevy
Github user eliaslevy commented on the issue: https://github.com/apache/flink/pull/6076 Thanks, that helps. Regarding debugging watermarks, at least in 1.4.2 sources did not display their emitted watermarks, although this may have been fixed in 1.5.0. In turn, that makes

[GitHub] flink issue #3334: FLINK-4810 Checkpoint Coordinator should fail ExecutionGr...

2018-05-27 Thread eliaslevy
Github user eliaslevy commented on the issue: https://github.com/apache/flink/pull/3334 Any chance this will be merged now that 1.5 is out? ---

[GitHub] flink pull request #6048: [FLINK-9403] [docs] Remove obsolete documentation

2018-05-20 Thread eliaslevy
GitHub user eliaslevy opened a pull request: https://github.com/apache/flink/pull/6048 [FLINK-9403] [docs] Remove obsolete documentation *Thank you very much for contributing to Apache Flink - we are happy that you want to help us improve Flink. To help the community review your

[GitHub] flink issue #4614: [FLINK-7547] AsyncFunction.scala extends Function, serial...

2017-08-29 Thread eliaslevy
Github user eliaslevy commented on the issue: https://github.com/apache/flink/pull/4614 👍 --- 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

[GitHub] flink issue #3477: [Flink-3318][cep] Add support for quantifiers to CEP's pa...

2017-04-28 Thread eliaslevy
Github user eliaslevy commented on the issue: https://github.com/apache/flink/pull/3477 I created [FLINK-6419](https://issues.apache.org/jira/browse/FLINK-6419). --- 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

[GitHub] flink issue #3477: [Flink-3318][cep] Add support for quantifiers to CEP's pa...

2017-04-15 Thread eliaslevy
Github user eliaslevy commented on the issue: https://github.com/apache/flink/pull/3477 Am I missing something or is there no way to get access to access to multiple events matched by these new quantifiers? The `PatternSelectFunction.select` takes an argument of `Map<Str

[GitHub] flink issue #2633: [FLINK-4497] CassadraSink support for scala tuples/case c...

2016-10-14 Thread eliaslevy
Github user eliaslevy commented on the issue: https://github.com/apache/flink/pull/2633 👍 --- 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

[GitHub] flink issue #2369: [FLINK-4035] Add a streaming connector for Apache Kafka 0...

2016-08-24 Thread eliaslevy
Github user eliaslevy commented on the issue: https://github.com/apache/flink/pull/2369 @rmetzger that's the one. NP. I realize breaking it up makes things easier. I just thought I'd mention it. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] flink issue #2369: [FLINK-4035] Add a streaming connector for Apache Kafka 0...

2016-08-23 Thread eliaslevy
Github user eliaslevy commented on the issue: https://github.com/apache/flink/pull/2369 This may be the wrong place to bring this up, but as you are discussing changes to the Kafka connector API, I think it is worth bring it up. As I've pointed out elsewhere, the current

[GitHub] flink issue #2108: [FLINK-4027] Flush FlinkKafkaProducer on checkpoints

2016-06-16 Thread eliaslevy
Github user eliaslevy commented on the issue: https://github.com/apache/flink/pull/2108 👍 --- 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

[GitHub] flink pull request #2108: [FLINK-4027] Flush FlinkKafkaProducer on checkpoin...

2016-06-15 Thread eliaslevy
Github user eliaslevy commented on a diff in the pull request: https://github.com/apache/flink/pull/2108#discussion_r67248839 --- Diff: flink-streaming-connectors/flink-connector-kafka-base/src/main/java/org/apache/flink/streaming/connectors/kafka/FlinkKafkaProducerBase.java

[GitHub] flink pull request #2108: [FLINK-4027] Flush FlinkKafkaProducer on checkpoin...

2016-06-15 Thread eliaslevy
Github user eliaslevy commented on a diff in the pull request: https://github.com/apache/flink/pull/2108#discussion_r67247830 --- Diff: flink-streaming-connectors/flink-connector-kafka-base/src/main/java/org/apache/flink/streaming/connectors/kafka/FlinkKafkaProducerBase.java