Github user asfgit closed the pull request at:
https://github.com/apache/flink/pull/1575
---
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
Github user vasia commented on the pull request:
https://github.com/apache/flink/pull/1575#issuecomment-199211011
I'll merge this today.
---
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 vasia commented on the pull request:
https://github.com/apache/flink/pull/1575#issuecomment-195477642
Anyone else wants to take a look at this PR or shall I go ahead and merge?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user vasia commented on the pull request:
https://github.com/apache/flink/pull/1575#issuecomment-194981424
Hey @s1ck,
thanks a lot for the valuable feedback. I've addressed your comments and
rebased on the current master. Regarding the method renamings, I suggest we
handle
Github user s1ck commented on the pull request:
https://github.com/apache/flink/pull/1575#issuecomment-194251517
Ok, I will keep that in mind. Thank you!
---
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
Github user vasia commented on the pull request:
https://github.com/apache/flink/pull/1575#issuecomment-194236417
Hey @s1ck,
thanks a lot for the thorough review! I will address your comments in the
following days. The PR is quite old and needs a rebase, too.
For future
Github user s1ck commented on the pull request:
https://github.com/apache/flink/pull/1575#issuecomment-193968765
Hi @vasia this is a nice PR, I commented some minor documentation / naming
issues
+1 for merging
---
If your project is set up for it, you can reply to this email and
GitHub user vasia opened a pull request:
https://github.com/apache/flink/pull/1575
[FLINK-3207] [gelly] adds the vertex-centric iteration abstraction
As per the mailing list discussion, this PR adds the vertex-centric /
pregel iteration abstraction to gelly. In this model,