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 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 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 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 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 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