Github user andralungu commented on the pull request:
https://github.com/apache/flink/pull/1105#issuecomment-139564327
The only argument I have for this particular code (as opposed to the one in
the DataSet API) is that his one simulates what Pregel is doing... But since
it's a library method, it should be used as is, without caring about the model.
IMO, the discussion in #1054 asked for this particular version, but if there
is a consensus I can change it to use the more efficient one... or even close
the PR and point people to the DataSet API...
---
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 enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
