[GitHub] flink pull request: [FLINK-3270] Add Kafka example

2016-02-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/1533 --- 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] flink pull request: [FLINK-3270] Add Kafka example

2016-02-11 Thread fhueske
Github user fhueske commented on the pull request: https://github.com/apache/flink/pull/1533#issuecomment-182779503 I think Robert has a good point. The examples should follow the implementations of the other examples after the rework in PR #1581. --- If your project is set up f

[GitHub] flink pull request: [FLINK-3270] Add Kafka example

2016-01-27 Thread rmetzger
Github user rmetzger commented on the pull request: https://github.com/apache/flink/pull/1533#issuecomment-175544829 I understand that argument, and I would be strongly against adding examples for each connector. The pull request is moving the example away from the Kafka proje

[GitHub] flink pull request: [FLINK-3270] Add Kafka example

2016-01-26 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/1533#issuecomment-174964868 Not quite sure about adding examples for the connectors to the examples project. If we do that for all connectors, it becomes heavily loaded with dependencies.

[GitHub] flink pull request: [FLINK-3270] Add Kafka example

2016-01-21 Thread rmetzger
GitHub user rmetzger opened a pull request: https://github.com/apache/flink/pull/1533 [FLINK-3270] Add Kafka example You can merge this pull request into a Git repository by running: $ git pull https://github.com/rmetzger/flink flink3270 Alternatively you can review and apply