[GitHub] flink pull request: [streaming] Properly forward rich window funct...

2015-06-25 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/855 --- 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 enabl

[GitHub] flink pull request: [streaming] Properly forward rich window funct...

2015-06-22 Thread mbalassi
Github user mbalassi commented on the pull request: https://github.com/apache/flink/pull/855#issuecomment-114179961 Merging. --- 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 enabl

[GitHub] flink pull request: [streaming] Properly forward rich window funct...

2015-06-22 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/855#issuecomment-114177459 Looks good, +1 to merge --- 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

[GitHub] flink pull request: [streaming] Properly forward rich window funct...

2015-06-22 Thread mbalassi
Github user mbalassi commented on the pull request: https://github.com/apache/flink/pull/855#issuecomment-114082949 Add reference to FLINK-2257 when merging. --- 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

[GitHub] flink pull request: [streaming] Properly forward rich window funct...

2015-06-22 Thread mbalassi
GitHub user mbalassi opened a pull request: https://github.com/apache/flink/pull/855 [streaming] Properly forward rich window function calls to wrapped fu… …nctions Open and close were not called, because of the wrapping in windowing. I should have realized this when fi