You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by vasia <gi...@git.apache.org> on 2015/12/17 19:09:49 UTC
[GitHub] flink pull request: [FLINK-3181] [gelly] avoid unnecessary message...
GitHub user vasia opened a pull request:
https://github.com/apache/flink/pull/1467
[FLINK-3181] [gelly] avoid unnecessary messages in SSSP examples and library method
I added a simple check in the messaging function to avoid sending unnecessary messages during the first superstep of vertex-centric SSSP. Initially, the workset contains all vertices, which means that all of them will execute the messaging function during the first iteration. However, only the source needs to generate messages in the first superstep; all other vertices would produce unnecessary messages with infinite-value distances.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vasia/flink FLINK-3181
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flink/pull/1467.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 #1467
----
commit 07525f4e0934b6421085999ac58fb5901e325554
Author: vasia <va...@apache.org>
Date: 2015-12-17T15:11:58Z
[FLINK-3181] [gelly] avoid unnecessary messages in SSSP examples and library method
----
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] flink pull request: [FLINK-3181] [gelly] avoid unnecessary message...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/flink/pull/1467
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] flink pull request: [FLINK-3181] [gelly] avoid unnecessary message...
Posted by tillrohrmann <gi...@git.apache.org>.
Github user tillrohrmann commented on the pull request:
https://github.com/apache/flink/pull/1467#issuecomment-166339189
Makes sense. +1 for 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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] flink pull request: [FLINK-3181] [gelly] avoid unnecessary message...
Posted by vasia <gi...@git.apache.org>.
Github user vasia commented on the pull request:
https://github.com/apache/flink/pull/1467#issuecomment-167564868
Thanks, I am merging this.
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---