Github user asfgit closed the pull request at:
https://github.com/apache/flink/pull/516
---
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 user mbalassi commented on the pull request:
https://github.com/apache/flink/pull/516#issuecomment-84936776
The integration for the scala AP is missing, other than that LGTM. Merging
it and adding the scala.
---
If your project is set up for it, you can reply to this email and
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/516#issuecomment-84879621
LGTM
---
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 an
Github user ggevay commented on the pull request:
https://github.com/apache/flink/pull/516#issuecomment-84743663
Yes, sorry for not providing a description.
I changed the return type of StreamExecutionEnvironment.execute() from void
to JobExecutionResult, and then added return stat
Github user hsaputra commented on the pull request:
https://github.com/apache/flink/pull/516#issuecomment-84741359
Will help review if more description about the direction of the patch
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user hsaputra commented on the pull request:
https://github.com/apache/flink/pull/516#issuecomment-84741309
Could give more description about how you go about with the PR?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
GitHub user ggevay opened a pull request:
https://github.com/apache/flink/pull/516
[FLINK-1767] [streaming] Make StreamExecutionEnvironment return
JobExecutionResult
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ggevay/flink J
