[GitHub] flink pull request: [FLINK-2191] Fix inconsistent use of closure c...

2015-06-10 Thread gyfora
Github user gyfora commented on the pull request: https://github.com/apache/flink/pull/813#issuecomment-110739702 Looks good to me :+1: --- 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

[GitHub] flink pull request: [FLINK-2191] Fix inconsistent use of closure c...

2015-06-10 Thread mbalassi
Github user mbalassi commented on a diff in the pull request: https://github.com/apache/flink/pull/813#discussion_r32102450 --- Diff: flink-staging/flink-streaming/flink-streaming-scala/src/main/scala/org/apache/flink/streaming/api/scala/DataStream.scala --- @@ -781,11 +780,24 @@

[GitHub] flink pull request: [FLINK-2191] Fix inconsistent use of closure c...

2015-06-10 Thread mbalassi
Github user mbalassi commented on a diff in the pull request: https://github.com/apache/flink/pull/813#discussion_r32102485 --- Diff: flink-staging/flink-streaming/flink-streaming-scala/src/main/scala/org/apache/flink/streaming/api/scala/StreamExecutionEnvironment.scala --- @@

[GitHub] flink pull request: [FLINK-2191] Fix inconsistent use of closure c...

2015-06-10 Thread mbalassi
Github user mbalassi commented on the pull request: https://github.com/apache/flink/pull/813#issuecomment-110786608 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

[GitHub] flink pull request: [FLINK-2191] Fix inconsistent use of closure c...

2015-06-10 Thread aljoscha
GitHub user aljoscha opened a pull request: https://github.com/apache/flink/pull/813 [FLINK-2191] Fix inconsistent use of closure cleaner in Scala Streaming The closure cleaner still cannot be disabled for the Timestamp extractor in Time and for the delta function in Delta

[GitHub] flink pull request: [FLINK-2191] Fix inconsistent use of closure c...

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