[GitHub] flink pull request: [FLINK-2780] Remove Old Windowing Logic and AP...

2015-10-08 Thread aljoscha
GitHub user aljoscha opened a pull request: https://github.com/apache/flink/pull/1242 [FLINK-2780] Remove Old Windowing Logic and API This rewrites the few examples and tests that are remaining using the new Windowing API. You can merge this pull request into a Git repository by r

[GitHub] flink pull request: [FLINK-2780] Remove Old Windowing Logic and AP...

2015-10-08 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/1242#issuecomment-146599121 This is pretty straightforward. If no one has any reservations against removing the old API (which has been reworked into the new API) I would go ahead. The

[GitHub] flink pull request: [FLINK-2780] Remove Old Windowing Logic and AP...

2015-10-08 Thread uce
Github user uce commented on the pull request: https://github.com/apache/flink/pull/1242#issuecomment-146608549 +1 to remove. I find it confusing as well. --- 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 do

[GitHub] flink pull request: [FLINK-2780] Remove Old Windowing Logic and AP...

2015-10-08 Thread rmetzger
Github user rmetzger commented on the pull request: https://github.com/apache/flink/pull/1242#issuecomment-146738873 +1 to get rid of the old API asap. --- 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

[GitHub] flink pull request: [FLINK-2780] Remove Old Windowing Logic and AP...

2015-10-09 Thread aljoscha
Github user aljoscha closed the pull request at: https://github.com/apache/flink/pull/1242 --- 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 en