[
https://issues.apache.org/jira/browse/FLINK-2113?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14564987#comment-14564987
]
ASF GitHub Bot commented on FLINK-2113:
---------------------------------------
GitHub user andralungu opened a pull request:
https://github.com/apache/flink/pull/749
[FLINK-2113][gelly] removed env.execute() after print()
This PR removes all the env.execute() that occur after a Dataset.print() in
gelly examples to fit the new functionality.
@uce can you have a quick look at this?
Thanks!:)
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/andralungu/flink print
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flink/pull/749.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #749
----
commit ad0069fe1a4b314aa571d094995d71bd2744ee34
Author: andralungu <[email protected]>
Date: 2015-05-29T15:41:51Z
[FLINK-2113][gelly] removed env.execute() after print()
----
> env.execute() is no longer needed after a call to print()
> ---------------------------------------------------------
>
> Key: FLINK-2113
> URL: https://issues.apache.org/jira/browse/FLINK-2113
> Project: Flink
> Issue Type: Bug
> Components: Gelly
> Affects Versions: 0.9
> Reporter: Andra Lungu
> Assignee: Andra Lungu
>
> print() is now an eager method ( like collect() or count() ). That means that
> calling "print()" immediately triggers the execution, and no "env.execute()"
> is required any more.
> Having env.execute() triggers an exception.
> The examples should be updated accordingly.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
